[U-Boot-Users] Patch to clean up syntax highlighting

[Jerry Van Baren]

Steven A. Falco wrote:
> My text-editor (vim) has a bit of trouble syntax-highlighting the cmd_nvedit.c
> file, because it apparently does not parse C ifdef/else/endif.  The following
> patch does not change the behavior of the code at all, but does allow the
> editor to properly syntax-highlight the file.
> 
> Comments invited.

OK, you invited comments so I'll be the designated curmudgeon tonight... 
I'm not wild about uglifying our code to accommodating vim syntax 
highlighting limitations.  Perhaps you can fix the vim syntax 
highlighting instead?

> Signed-off-by: Steve Falco <sfalco at harris.com>
> 
> diff --git a/common/cmd_nvedit.c b/common/cmd_nvedit.c
> index 9c5d1fc..1ac91ea 100644
> --- a/common/cmd_nvedit.c
> +++ b/common/cmd_nvedit.c
> @@ -179,11 +179,12 @@ int _do_setenv (int flag, int argc, char *argv[])
>  		 * Ethernet Address and serial# can be set only once,
>  		 * ver is readonly.
>  		 */
> +		if (
>  #ifdef CONFIG_HAS_UID
>  		/* Allow serial# forced overwrite with 0xdeaf4add flag */
> -		if ( ((strcmp (name, "serial#") == 0) && (flag != 0xdeaf4add)) ||
> +		    ((strcmp (name, "serial#") == 0) && (flag != 0xdeaf4add)) ||
>  #else
> -		if ( (strcmp (name, "serial#") == 0) ||
> +		    (strcmp (name, "serial#") == 0) ||
>  #endif
>  		    ((strcmp (name, "ethaddr") == 0)
>  #if defined(CONFIG_OVERWRITE_ETHADDR_ONCE) && defined(CONFIG_ETHADDR)

I probably should confess I don't use syntax highlighting so that 
probably factors in to my lack of sympathy.

Best regards,
gvb