[U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch

[Mike Frysinger]

On Friday, April 15, 2011 04:25:58 Macpaul Lin wrote:
> 2011/4/15 Mike Frysinger:
> > On Friday, April 15, 2011 03:02:58 Macpaul Lin wrote:
> >>  #if !defined(CONFIG_ENV_IS_IN_EEPROM)        && \
> >> -    !defined(CONFIG_ENV_IS_IN_FLASH) && \
> >> +     !defined(CONFIG_ENV_IS_IN_FLASH)        && \
> > 
> > not sure this one hunk is desirable, but the rest are fine
> > -mike
> 
> Do you mean the replacement with space to ident?
> 
> According to the 2.6.38.1's checkpatch, it reported:
> WARNING: please, no spaces at the start of a line

this particular check has already been posted upstream as generally causing 
more harm than good.  but getting checkpatch fixed often feels glacial.

up to Wolfgang how he feels about ifdef indentation
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20110415/a00ad3df/attachment.pgp