<table><tr><td style="">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/D13900">View Revision</a></tr></table><br /><div><div><p><a href="https://phabricator.kde.org/p/kfunk/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@kfunk</a> Ping? Do you agree those code snippets will not see a quick normal fix, so the current warnings filling the logs serve no good?</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/D13900#inline-73429">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aaronpuchert</span> wrote in <span style="color: #4b4d51; font-weight: bold;">qmljsparser.cpp:51-53</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">If that is true, the change should be made somewhere else.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Well spotted, had missed myself. before. Having looked more at it, this file is part of the snapshot copied over from Qt sources, and only regenerated over there during Qt development, at least so far not in the forked code copy here in KDevelop.</p>

<p style="padding: 0; margin: 8px;">How to test applying this ifdef to qmljs.g I have to leave to those who know about the QML code copy.<br />
Myself I would for now just apply to the generated code, to get rid of the rather useless warning noise hiding real issues.</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/D13900">https://phabricator.kde.org/D13900</a></div></div><br /><div><strong>To: </strong>kossebau, KDevelop<br /><strong>Cc: </strong>aaronpuchert, kfunk, kdevelop-devel, antismap, iodelay, vbspam, njensen, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd<br /></div>