> > usr/src/uts/common/io/dls/dls_link.c
 > > ------------------------------------
 > > 
 > > * 901-906: I thought that this kind of thing would generate lint errors
 > >   for non-debug builds, as you end up with an empty block.  In any case,
 > >   this can also be written as:
 > > 
 > >   ASSERT(dlp->dl_mh == NULL || dlp->dl_mh == mh);
 > >   if (dlp->dl_mh == NULL) {
 > >       dlp->dl_mh = mh;
 > >       dlp->dl_mip = mac_info(mp);
 > >   }
 > > 
 > Accept.
 > 
 > While I can do what you suggest. Is this (lint error for non-debug builds) 
 > really matters?

No, it doesn't.  In fact, we explicitly don't care about lint errors in
non-DEBUG builds.

--
meem

Reply via email to