[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