On Fri, 12 Mar 2010 21:20:05 +0000 Dave wrote: DS> > The stderr log is enabled if a message is logged before any log handlers are DS> > enabled: DS> DS> > The problem is, it is not disabled if a log handler is added. DS> DS> That's definitely not sensible. DS> DS> If logging is explicitly specified, then this is what ought to be used. DS> Anything that's active in advance of this, should be regarded as a DS> temporary stopgap.
Yep. My thoughts exactly. DS> - I noticed that cert_util.c seems to be printing directly to stderr DS> (immediately above the deleted line). I haven't looked to see DS> the context of this message, but that feels probably wrong. DS> Certainly worth a closer look. That's inside an ifdef for a main() function for testing, so it won't show up in the library. DS> - I'd suggest that the message when switching away from the DS> temporary logging should perhaps be at a lower severity DS> (maybe INFO rather than WARNING?). Ok, I'll change that. Since there have been no objections, I'll probably commit the patch in the next few days. ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev _______________________________________________ Net-snmp-coders mailing list Net-snmp-coders@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
