On Fri, Feb 12, 2010 at 11:28:27AM -0800, Wes Hardaker wrote: > >>>>> On Wed, 10 Feb 2010 07:49:57 +0100, Magnus Fromreide > >>>>> <ma...@lysator.liu.se> said: > > MF> We do not really need the <oidName>_len variables as it is enough to > MF> add the length to the declarations and then use OID_LENGTH. > > That's very much a style issue... Some argue that it looks cleaner to > use _len variables, others don't mind seeing the OID_LENGTH() > invocations.
Agreed. > I actually went through the same thought process when I created the > variable. But I ended up leaving it "like the others were done" to keep > it consistent. > > (a good optimizer should optimize it out, but.....) I want one of those - one that manages to replace a non-const variable in a (shared) library with the proper constant value in the final program after it have proved that the value really is const. /MF ------------------------------------------------------------------------------ SOLARIS 10 is the OS for Data Centers - provides features such as DTrace, Predictive Self Healing and Award Winning ZFS. Get Solaris 10 NOW http://p.sf.net/sfu/solaris-dev2dev _______________________________________________ Net-snmp-coders mailing list Net-snmp-coders@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
