Hi Michael,

On Fri, May 11, 2012 at 5:50 PM, Michael Walle <mich...@walle.cc> wrote:
> Future compiler versions may generate a "too many arguments for functions"
> warning.
>
> Signed-off-by: Michael Walle <mich...@walle.cc>
> Cc: Joe Hershberger <joe.hershber...@gmail.com>
> ---
>  net/eth.c |   14 ++++++++++++--
>  1 files changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/net/eth.c b/net/eth.c
> index c9f05d8..afce863 100644
> --- a/net/eth.c
> +++ b/net/eth.c
> @@ -58,7 +58,12 @@ int eth_getenv_enetaddr_by_index(const char *base_name, 
> int index,
>                                 uchar *enetaddr)
>  {
>        char enetvar[32];
> -       sprintf(enetvar, index ? "%s%daddr" : "%saddr", base_name, index);
> +
> +       if (index)
> +               sprintf(enetvar, "%s%daddr", base_name, index);
> +       else
> +               sprintf(enetvar, "%saddr", base_name);
> +
>        return eth_getenv_enetaddr(enetvar, enetaddr);
>  }
>
> @@ -66,7 +71,12 @@ static int eth_mac_skip(int index)
>  {
>        char enetvar[15];
>        char *skip_state;
> -       sprintf(enetvar, index ? "eth%dmacskip" : "ethmacskip", index);
> +
> +       if (index)
> +               sprintf(enetvar, "eth%dmacskip", index);
> +       else
> +               sprintf(enetvar, "ethmacskip");
> +
>        return ((skip_state = getenv(enetvar)) != NULL);
>  }

This seems like this could improve by consolidating the logic to come
up with the ethaddr variable name in an inline function.  Especially
since you are about to add another use in the setenv case.  Do you
agree, Mike?  Or is 3 uses of a relatively simple algorithm better
inlined than separated?

-Joe
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to