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

John Baldwin jhb at freebsd.org
Fri Aug 4 15:36:05 UTC 2006


On Friday 04 August 2006 03:46, Yar Tikhiy wrote:
> 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.

If you have INVARIANTS on then every mtx_lock/unlock will include __LINE__ and 
__FILE__.

-- 
John Baldwin


More information about the cvs-src mailing list