[U-Boot] [PATCH v0 1/4] Groundwork for generalization of env interface
Wolfgang Denk
wd at denx.de
Mon Nov 7 23:05:05 CET 2011
Dear Gerlando Falauto,
In message <4EB84859.6000906 at keymile.com> you wrote:
>
> -int _do_env_set (int flag, int argc, char * const argv[])
> +int env_check_apply(const char *name, const char *oldval,
> + const char *newval, int flag)
>
> > Please use only TAB for indentation. Please fix globally.
>
> From fs/ubibfs/ubifs.h:
Never ever use examples from other code to argument your's was right -
the example you chose might be wrong as well.
> Could you please provide some examples as to what would be the correct
> coding style for function declarations and/or function calls that spawn
> on multiple lines? I could not find anything on the topic.
http://www.denx.de/wiki/U-Boot/CodingStyle:
Use TAB characters for indentation and vertical alignment, not
spaces
> + if (himport_ex(&env_htab, (char *)default_environment,
> + sizeof(default_environment), '\0', 0,
> + 0, NULL, apply_function) == 0) {
>
> What should be the right indentation?
In any case it makse no sense to have the 2nd and 3rd line indented
differently, right?
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"A child is a person who can't understand why someone would give away
a perfectly good kitten." - Doug Larson
More information about the U-Boot
mailing list