cvs commit: src/sys/dev/em if_em.c if_em.h
yar at comp.chem.msu.su
Fri Aug 4 07:47:01 UTC 2006
On Fri, Aug 04, 2006 at 09:23:23AM +0200, Stefan Farfeleder wrote:
> On Fri, Aug 04, 2006 at 11:19:09AM +0400, Yar Tikhiy wrote:
> > As a general remark, it is a very good idea to keep large style(9)
> > sweeps separate from actual code changes even if they are tiny.
> > Changing style(9) shouldn't alter the binary code produced, and
> > this can be 100% verified using cmp(1), diff(1), cksum(1), or md5(1).
> > Of course, changing line numbers will affect debug info, but the
> > latter can be strip(1)'ed. In the case under discussion, I'd compare
> > if_em.o with its former version to make sure no code changed.
> It will produce different code if __LINE__ is used.
Yeah, it's a thing to watch out for. As are __TIME__ and __DATE__.
Fortunately, they are not widely used in src/, AFAIK.
More information about the cvs-src