<table><tr><td style="">aaronpuchert 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/D14694">View Revision</a></tr></table><br /><div><div><p>Maybe this has been discussed before, but I think something like a magnifying glass (<tt style="background: #ebebeb; font-size: 13px;">search</tt>) would be more appropriate. The check mark suggests that the code is "verified", in the sense that it's definitely fine, but most (all?) static analysis tools don't guarantee that. If the tool finds something, then something is probably wrong, but if it doesn't, there could still be a mistake.</p>

<p>The tools also get better, adding more checks, which means that there might be no issues with some code first, but on updating the analysis tool new issues appear. Then it's weird to have the check mark, because once "it worked" and now "it doesn't". With the magnifying glass it's clear that we search for issues, and with new versions of our tool we might find more.</p></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/D14694">https://phabricator.kde.org/D14694</a></div></div><br /><div><strong>To: </strong>kossebau, KDevelop<br /><strong>Cc: </strong>aaronpuchert, kdevelop-devel, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd<br /></div>