Hello there,

Offending source code is

      if(  ( BattVoltage> 129 ) || ( BattVoltage < 144 ) )

Maybe swap || for && would be better.

This problem I found by using cppcheck. It said

[rhino.c:190]: (warning) Logical disjunction always evaluates to true: 
BattVoltage> 129 || BattVoltage < 144.

Regards

David Binderman                                           
_______________________________________________
Nut-upsdev mailing list
[email protected]
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/nut-upsdev

Reply via email to