On Fri, 08 Dec 2006 06:03:59 -0700 Rocky Craig <[EMAIL PROTECTED]> wrote:
> Andrew Morton wrote: > > >In fact, please don't write macros. > > > > > I see them all over the place (i.e., EXPORT_SYMBOL) so the request > is a little confusing to me... It's a general principle. Sometimes they're unavoidable. Other times they are avoidable but people add them anyway. > > Please don't write macros which require that the caller have a particular > > > >local variable of a particular name. > > > > > I could understand that, even if some people did agree with my desire > to increase legibility by decreasing visual clutter. I think it also > provides > some protection against typos in argument lists. > > I'm (naively) curious as to why it's being flagged now as opposed to > two years ago when I submitted the original additions. It probably didn't get noticed. Some of the stuff we have in there is unbelieveable. > Have reviewers or review methods changed? > > Have standards changed? We have become more vigilant over time. > Did it just slip under someone's radar two years ago? I'd say so, yes. ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Openipmi-developer mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openipmi-developer
