cvs commit: src/sys/dev/em if_em.c if_em.h

Yar Tikhiy yar at
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 mailing list