Kleine Warning in 10_CUL_HM

[mcfly71]

Hallo Leute,

ich wollte nur kurz darauf hinweisen:
Im Script 10_CUL_HM ist bei der VD Hänger Routine ein kleiner
Fehler drin, den mein Perl immer ausgibt:

#VD hang detection
 my $des = ReadingsVal($name, "ValveDesired", "");
 $des =~ s/ .*//; # remove unit
 if (defined $des && ($des ne $vp) && ($err&0x30) == 0x00){


Fehlermeldung:
Argument "" isn't numeric in numeric ne (!=)

$des oder $vp kann anscheinend auch leer sein und deshalb ist wohl besser ein ne statt des != ???!!!!
Würde das derjenige bitte einchecken (wenn es kein Fehler anderer Natur ist, sowas wie: Da darf keine Variable eigentlich leer sein....

mcfly

[mcfly71]

im original steht natürlich noch nicht das ne drin, sondern ein !=

original:
if (defined $des && $des != $vp && ($err&0x30) == 0x00){

mcfly

[martinp876]

mcfly
 bau ich ein. Kann noch etwas einfacher sein, $des existiert in jeden Fall
     if ($des ne $vp && ($err&0x30) == 0x00){
Gruss Martin