Messages in this thread |  | | Subject | Re: [PATCH] Staging: speakup: synth.c: removed a space | From | Joe Perches <> | Date | Tue, 03 Dec 2013 23:45:15 -0800 |
| |
On Wed, 2013-12-04 at 10:38 +0300, Dan Carpenter wrote: > On Wed, Dec 04, 2013 at 06:35:15AM +0200, Aldo Iljazi wrote: > > Samuel Thibault wrote: > > > > > Err, I'd rather make it really visible that the for loop doesn't have > > > its first statement? > > > > Wouldn't it be better if you add a comment there? So it would follow the > > coding style? > > No. Adding obvious comments is more annoying than the space. > > This seems like a small bug in checkpatch.pl. Joe, the problem is this > code: > > for ( ; x; x++) { > > It's complaining about the space character before the semicolon.
Shrug. checkpatch isn't and can't be perfect.
I think it's for things like
for (;;)
and that's better than
for ( ; ; ) or for ( ;; ) or for ( ;;)
|  |