2008/6/26 Garrett D'Amore <[EMAIL PROTECTED]>: > I have put together a webrev with fixes to address lint warnings > generated thanks to MBLKL and company being lint-unclean. They are now > lint clean. Additionally, I did related lint abd whitespace (cstyle) in > a bunch of drivers. Some of the drivers I was even able to remove all > of the LINTTAGS overrides, yielding truly lint clean source! (I could > have done more of the drivers, but the effort to clean *all* of uts up > is rather larger than I want to take on right now.) I also fixed the > fact that one driver (amd8111s) was pointlessly including a dependency > upon drv/ip. (I've verified that the link dependency is not necessary, > both by manual examination of the symbol table, and by loading the > generated object without the dependency on a system.) > > The webrev is at > > http://cr.opensolaris.org/~gdamore/mblkl
Garret, some time ago I found a bug in cstyle and came up with a proposed fix for it. I don't know if it applies to your work or not, but I thought you should know: 6609055 cstyle erroneously reports whitepace before right paren for some "for loops" http://cr.opensolaris.org/~swalker/4754400/usr/src/tools/scripts/cstyle.pl.wdiff.html It's still on the "awaiting sponsor" list :-) Cheers, -- Shawn Walker _______________________________________________ opensolaris-code mailing list opensolaris-code@opensolaris.org http://mail.opensolaris.org/mailman/listinfo/opensolaris-code
