Similar to the message earlier today, the following changes look like
they could be made in the snmp_bc plugin to address issue #1804512,
"Debug output occurring during normal operation".  Please let me know if
you agree and want me to make these changes (or whether you'd prefer to
do so).  For these, I know that they're in "#if 0" code, so it probably
doesn't matter too much, unless someone changes the #if in the future.

In addition, I have a question about snmp_bc_sel.c, line 586.  Looking
at other snmp_bc_snmp_get calls, it appears that a return value which is
not SA_OK is indeed an error (and the err() macro is used), but the code
falls through at this point.  Is that intentional?  From what I can see,
an error is reported, but then execution continues.

-- 
Bryan Sutula <[EMAIL PROTECTED]>

=======================================================================
--- svn/SF/openhpi/plugins/snmp_bc/snmp_bc_sensor.c     2007-10-03 
15:25:20.000000000 -0600
+++ sandbox/openhpi/plugins/snmp_bc/snmp_bc_sensor.c    2007-11-05 
12:10:56.000000000 -0700
@@ -153,12 +153,12 @@
        {       /* Debug section */
                SaHpiTextBufferT buffer;
 
-               err("Sensor=%s", rdr->IdString.Data);
+               dbg("Sensor=%s", rdr->IdString.Data);
                oh_decode_sensorreading(working_reading, 
rdr->RdrTypeUnion.SensorRec.DataFormat, &buffer);
-               err("  Reading: %s.", buffer.Data);
+               dbg("  Reading: %s.", buffer.Data);
 
                oh_decode_eventstate(working_state, 
rdr->RdrTypeUnion.SensorRec.Category, &buffer);
-               err("  Event State: %s\n", buffer.Data);
+               dbg("  Event State: %s\n", buffer.Data);
        }
 #endif


-------------------------------------------------------------------------
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/
_______________________________________________
Openhpi-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openhpi-devel

Reply via email to