Re: [Qemu-devel] Bug in checkpatch.pl?

2011-08-26 Thread Stefan Hajnoczi
On Fri, Aug 26, 2011 at 11:28 AM, Pavel Borzenkov wrote: > checkpatch.pl correctly reports a warning for the following code: > > if (something) >    foo; > else >    bar; > > WARNING: braces {} are necessary for all arms of this statement > #1: FILE: tmp/a.c:1: > +if (something) > [...] > +else >

[Qemu-devel] Bug in checkpatch.pl?

2011-08-26 Thread Pavel Borzenkov
Hi. checkpatch.pl correctly reports a warning for the following code: if (something) foo; else bar; WARNING: braces {} are necessary for all arms of this statement #1: FILE: tmp/a.c:1: +if (something) [...] +else [...] But if I modify it as follows, checkpatch.pl doesn't report any w