<div class="gmail_quote">On Sun, May 13, 2012 at 10:49 AM, David Edmundson <span dir="ltr"><<a href="mailto:david@davidedmundson.co.uk" target="_blank">david@davidedmundson.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Martin and I have been fixing lots of issues from Krazy the KDE source<br>
code checker.<br></blockquote><div><br></div><div>We set a goal to have all Krazy checks pass for 0.4 and we're almost done \o/</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br>
One of them I think is a false-positive. I.e Krazy is telling us we're<br>
doing something wrong when actually we're not.<br>
<br>
<a href="http://www.englishbreakfastnetwork.org/krazy/reports/extragear/network/telepathy/index.html" target="_blank">http://www.englishbreakfastnetwork.org/krazy/reports/extragear/network/telepathy/index.html</a><br>
<br>
"ktp-kded-module/contact-request-handler.cpp: line# 190 (1)<br>
A few Qt methods should be avoided because they don't work as expected<br>
or because you need to really be an expert to use them correctly.<br>
<a href="http://techbase.kde.org/Policies/API_to_Avoid" target="_blank">http://techbase.kde.org/Policies/API_to_Avoid</a> for more information."<br></blockquote><div><br></div><div><div>Another false positives are the header guards in ktp-accounts-kcm - the plugins do have a proper header guard, it's just ridiculously long (ie. #ifndef KCMTELEPATHYACCOUNTS_PLUGIN_HAZE_MYSPACEIM_ACCOUNT_PARAMETERS_WIDGET_H).</div>

<div><br></div><div>This looks like another Krazy bug to me, but I think we could change it anyway (sed out the KCMTE..._PLUGIN_). Comments?</div></div><div><br></div></div>--<div><font color="#666666">Martin Klapetek | KDE Developer</font></div>

<br>