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 <[EMAIL PROTECTED]> > > 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 ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
