Hello.
I propose that we do one of the following:
a) remove the DEBUGINDENT macro from the headers.
b) change the define of DEBUGINDENT to always be
#define DEBUGINDENT()
Motivation:
It doesn't work at all.
Currently the definition is
#define DEBUGINDENT() do {if (_DBG_IF_) {__DBGINDENT();} }while(0)
__DBGINDENT() is also a define that expands to debug_indent().
debug_indent() is a pure function (no side effects, specifically no
output) so that could be removed.
Now we are down to
do { _DBG_IF_; }while(0)
_DBG_IF_ expands to snmp_get_do_debugging() and that is also a pure
function, thus we are down to
do { } while(0)
and that is trivially dead code.
-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems? Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Net-snmp-coders mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
