<table><tr><td style="">tcanabrava 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/D26129">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D26129#582336" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D26129#582336</a>, <a href="https://phabricator.kde.org/p/ervin/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ervin</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>The patch being large I didn't check each one separately but I suspect this patch misses boat loads of qAsConst.</p></div>
</blockquote>

<p>Yes, it indeed misses, I'll add the qAsConst. ( I would actually prefer to change the QList to std::vector in the future, no need to add the 'qAsConst' there and it would help readability )</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Besides there's at least one case of very flawed logic, it can't be about blindly switching one loop construct for another especially when the iterators are compared inside the loop body.</p></blockquote>

<p>I didn't blindly change anything, everything I did I tested either by running the unittests or recompilling and testing the settings of applications.</p>

<p>This patch greatly increases the readability of the software. I'll do another round of reviews here adding the qAsConst and trying to find broken logic but I don't believe they are really broken.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R237 KConfig</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26129">https://phabricator.kde.org/D26129</a></div></div><br /><div><strong>To: </strong>tcanabrava, ervin<br /><strong>Cc: </strong>ervin, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>