[cfe-commits] [PATCH][PR11329] StaticAnalyzer: Warn when statement body is a semicolon -- an unintended null statement

Dmitri Gribenko gribozavr at gmail.com
Wed Jan 4 05:00:26 PST 2012


On Wed, Jan 4, 2012 at 1:09 AM, Nico Weber <thakis at chromium.org> wrote:
> So if you add this, you might want to put it under that flag.

OK.

> But
> since that flag exists already, I'd guess that whoever added that
> tried adding this for loops as well and decided that the noise level
> is too high. It might be worth to look at the svn history of that
> warning.

I have just checked, but didn't find any evidence of that.

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/



More information about the cfe-commits mailing list