<table><tr><td style="">dfaure updated this revision to Diff 72814.<br />dfaure retitled this revision from "ki18n: add unittest for KUitSetup::setFormatForMarker" to "ki18n: export Kuit::setupForDomain, unittest KUitSetup::setFormatForMarker".<br />dfaure edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-ok5nhjvocs5y7sm/">(Show Details)</a><br />dfaure added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D26408">View Revision</a></tr></table><br /><div><div><p>Update commit log</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">This <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">is</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">was initially</span> to test the port to QRegularExpression in D26366<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">.<br />
<br />
HOWEVER setFormatForMarker wasn't used anywhere (part of a class</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">,</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">whose getter Kuit::</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">it turned out that </span>setupForDomain was<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">n't</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> meant to be</span> exported<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">).</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">So the alternative</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">since</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">would be to just remove it, as well as formatForMarker(), no?</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">KUitSetup was exported.</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R249 KI18n</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D26408?vs=72722&id=72814">https://phabricator.kde.org/D26408?vs=72722&id=72814</a></div></div><br /><div><strong>BRANCH</strong><div><div>2020_01_add_unittest</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26408">https://phabricator.kde.org/D26408</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/klocalizedstringtest.cpp<br />
autotests/klocalizedstringtest.h<br />
src/kuitmarkup.h</div></div></div><br /><div><strong>To: </strong>dfaure, ilic<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>