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 ( ;;)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
