Hi Silvino,<br><br>could you please check the below thread?<br>Your lights on this would be much appreciated.<br><br>thanks,<br>Arnaud<br><br><div class="gmail_quote">2011/11/3 Thomas Jarosch <span dir="ltr"><<a href="mailto:thomas.jarosch@intra2net.com">thomas.jarosch@intra2net.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">On Saturday, 15. October 2011 09:10:18 Arnaud Quette wrote:<br>
> >>> Detected by "cppcheck", not tested on real hardware.<br>
> >>><br>
> >>> -      if(  ( BattVoltage > 129 ) || ( BattVoltage < 144 ) )<br>
> >>> +      if(  ( BattVoltage > 129 ) && ( BattVoltage < 144 ) )<br>
> >><br>
> >>   I think the current, ||, logic is fine.  I can't see in what<br>
> >> circumstances it would yield the wrong decision.  Can anyone point a<br>
> >> scenario where I am wrong?<br>
> ><br>
> > As the diagnostic says, it is always true.  No matter what value<br>
> > BattVoltage takes, whether float or int,<br>
> > it is always either > 129 or < 144.   It is clearly a bug, but we<br>
> > aren't sure whether it should be<br>
> > in range, out of range, or just remove the test altogether (since it<br>
> > seems to work when always true).<br>
><br>
> the best thing to do, in such case, remains to contact the author.<br>
><br>
> @Silvino: could you please check this thread and give us back your<br>
> opinion? <a href="http://blog.gmane.org/gmane.comp.monitoring.nut.devel" target="_blank">http://blog.gmane.org/gmane.comp.monitoring.nut.devel</a><br>
<br>
</div>ping.<br>
<br>
Cheers,<br>
Thomas<br>
</blockquote></div><br>