<div class="gmail_quote">On Thu, May 3, 2012 at 1:51 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



Hi Richard,<div><br></div><div>if you're looking for data if this should be on by default: This has 0 false positives in all of chromium (including tests, all dependencies) and found 0 bugs.</div></blockquote><div><br>



</div><div>Thanks for the data, Nico.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><br></div><div>(I did put in a true positive ("for (int i; i < 5;) {}") somewhere and verified that that got detected, to make sure I enabled the warning correctly. My first attempt to introduce a true positive, "int i; while (i > 0) {}" didn't fire, I suppose while loops aren't handled intentionally?)</div>



</blockquote><div><br></div><div>While and do-while loops aren't covered due to false positives, such as this code:</div><div><br></div><div>void Run(int times) {</div><div>  while (times == -1) DoStuff();</div><div>


  for (int i = 0; i< times; i++) DoStuff();</div><div>}</div><div><br></div><div>There's also some macros that seems to trigger this warning that I'm investigating.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span><font color="#888888">
<div><br></div></font></span><div><span><font color="#888888">Nico</font></span><div><div><br></div></div></div></blockquote></div>