<table><tr><td style="">kossebau marked an inline comment as done.<br />kossebau 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/D8512" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>So as discussed on irc, will not discard this then, but see to turn this into something all maintainers agree on (and see how to get stringhelper.cpp solved)</p>

<p>Will prepare next some Best Practices Rules for qt string handling code (whether using QStringLiteral or QLatin1String in qstringbuilder-powered concatenations, how to avoid .data duplication when using same qstringliterals, etc) to put in some CODING,md file, so we have some golden rules around which should help to reduce the current random approaches. If good enough perhaps also could be moved to kde wikis for shared usage.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D8512#inline-36725" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">kfunk</span> wrote in <span style="color: #4b4d51; font-weight: bold;">CMakeLists.txt:1</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Is this needed? Isn't it set by the parent CMakeLists.txt?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Good catch. Forgot to remove again, left-over from incrementally extending setting those defs.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8512" rel="noreferrer">https://phabricator.kde.org/D8512</a></div></div><br /><div><strong>To: </strong>kossebau<br /><strong>Cc: </strong>kfunk, kdevelop-devel<br /></div>