[U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch
    Graeme Russ 
    graeme.russ at gmail.com
       
    Sat Apr 16 09:07:06 CEST 2011
    
    
  
On 16/04/11 16:22, Albert ARIBAUD wrote:
> Hi all,
> 
> Le 15/04/2011 12:09, Wolfgang Denk a écrit :
>> Dear Mike Frysinger,
>>
>> In message<201104150453.21441.vapier at gentoo.org>  you wrote:
>>>
>>>>> 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)&&  \
>>>>>
[snip]
> 
> If the goal is to have the !defined aligned, then maybe a solution here 
> would be toput a tab rather than a space after the #if?
> 
I agree - Aligned looks cleaner and easier to read (especially if the logic
is multi-levelled) and shutting up checkpatch by using a tab rather than a
space is an easy solution
But, fwiw, once the patch is applied and everything is nicely aligned and
reads well, who's going to care that checkpatch complained?
Regards,
Graeme
    
    
More information about the U-Boot
mailing list