On Mon, Apr 1, 2013 at 3:54 PM, Zdenek Styblik <zdenek.styb...@gmail.com> wrote:
> Dan,
>
> here is what I see wrong with this patch:
>
> * it touches/modifies more than just help
> * ``printf("Sensor Type %s not found.\n", type);'' -> ``lprintf(LOG_ERR, 
> ...);''
>
> I find the following, despite not covered/modified by the patch, to be
> disputable as well: ``printf("Sensor Types:\n");''. I think this
> should go to ``lprintf(LOG_NOTICE, ...);''. Just a note/observation.
>

So just change these:
                if (sensor_type != x) {
+                       printf("Sensor Type %s not found.\n", type);
                        printf("Sensor Types:\n");

to use lprintf(LOG_NOTICE, ..)?

Ok, no problem.. I'll fix this.

thanks
dan

------------------------------------------------------------------------------
Own the Future-Intel&reg; Level Up Game Demo Contest 2013
Rise to greatness in Intel's independent game demo contest.
Compete for recognition, cash, and the chance to get your game 
on Steam. $5K grand prize plus 10 genre and skill prizes. 
Submit your demo by 6/6/13. http://p.sf.net/sfu/intel_levelupd2d
_______________________________________________
Ipmitool-devel mailing list
Ipmitool-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ipmitool-devel

Reply via email to