[U-Boot] what is with the "#if 0"ed part of ARP hdr in include/net.h?

Wolfgang Denk wd at denx.de
Mon Jul 18 21:24:01 CEST 2016


Dear "Robert P. J. Day",

In message <alpine.LFD.2.20.1607170606270.5540 at localhost.localdomain> you wrote:
> 
>   was poking around in include/net.h, and noticed this snippet:
...
>   is it doing anything, like acting as a comment or something?

Probably. Sorry, I dont remember the details.

> p.s. there's quite a lot of "#if 0" throughout the source code, some
> of which i may submit patches to get rid of just in the areas i'm
> confortable with.

In many cases this was used to create "minimal" diffs compared to the
original code imported from elsewhere.  Commenting a bigger block of
code using "#if 0" was deemed to be easier to forward-port to new
versions of the code than deleting the whole block, where any changes
in this (uninteresting) code would cause conflicts.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
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
If the car industry behaved like the computer industry over the  last
30  years, a Rolls-Royce would cost $5, get 300 miles per gallon, and
blow up once a year killing all passengers inside.


More information about the U-Boot mailing list