[U-Boot] [PATCH v0 1/4] Groundwork for generalization of env interface

Wolfgang Denk wd at denx.de
Sat Nov 5 17:09:58 CET 2011


Dear Gerlando Falauto,

In message <1319647072-17504-2-git-send-email-gerlando.falauto at keymile.com> you wrote:
> Signed-off-by: Gerlando Falauto <gerlando.falauto at keymile.com>
> ---
>  common/cmd_nvedit.c   |  152 +++++++++++++++++++++++++++++++------------------
>  common/env_common.c   |   15 ++++-
>  include/environment.h |    7 ++
>  include/search.h      |   13 ++++
>  lib/hashtable.c       |   50 ++++++++++++++++
>  5 files changed, 179 insertions(+), 58 deletions(-)
...
> -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.


> -	if (ep) {		/* variable exists */
> +	if ((oldval != NULL) /* variable exists */
> +	 && ((flag & H_FORCE) == 0)) { /* and we are not forced */

Incorrect indentation.


> +/*
> + * Set a new environment variable,
> + * or replace or delete an existing one.
> +*/

Incorrect multiline comment style.

> +	/* Perform requested checks. Notice how since we are overwriting
> +	 * a single variable, we need to set H_NOCLEAR */

Incorrect multiline comment style.

>  void set_default_env(const char *s)
>  {
> +	/* By default, do not apply changes as they will eventually
> +	 * be applied by someone else */

Incorrect multiline comment style.

Please fix globally!

> +	if (himport_ex(&env_htab, (char *)default_environment,
> +		    sizeof(default_environment), '\0', 0,
> +		      0, NULL, apply_function) == 0) {

Incorrect / inconsistent indentation.  Please fix globally.


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
Applying computer technology is simply finding the  right  wrench  to
pound in the correct screw.


More information about the U-Boot mailing list