[U-Boot] [PATCH] net: Add labels to endifs and remove impossible conditional code block
Wolfgang Denk
wd at denx.de
Thu Oct 21 22:46:26 CEST 2010
Dear Reinhard Meyer,
In message <4CC0A479.4040900 at emk-elektronik.de> you wrote:
>
> >> -#if defined(CONFIG_CMD_DHCP)
> >> - *e++ = 53; /* DHCP Message Type */
> >> - *e++ = 1;
> >> - *e++ = DHCP_DISCOVER;
> >> -
> >> - *e++ = 57; /* Maximum DHCP Message Size */
> >> - *e++ = 2;
> >> - *e++ = (576 - 312 + OPT_SIZE)>> 16;
> >> - *e++ = (576 - 312 + OPT_SIZE)& 0xff;
> >> -#endif
> >
> > What makes you think CONFIG_CMD_DHCP was never used? There are 270+
> > boards that use this!
>
> Its in the ELSE part like this:
Ouch.
> #if defined(CONFIG_CMD_DHCP)
> ...
> #else
> ...
> #if defined(CONFIG_CMD_DHCP)
> ...
> #endif
> ...
> #endif
>
> Perhaps it should be investigated how it happened to become such dead code?
Indeed. This cannot be right / intentional.
Just deleting it seems wrong in any case.
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
People are very flexible and learn to adjust to strange surroundings
-- they can become accustomed to read Lisp and Fortran programs, for
example. - Leon Sterling and Ehud Shapiro, Art of Prolog, MIT Press
More information about the U-Boot
mailing list