Bugfix-Vorschlag 71_YAMAHA_AVR.pm - volumeStraight

[knopf_piano]

Hi Markus,
Fehler im Modul in der regex für volumeStraight-Prüfung.
Version

Code Auswählen Erweitern


# $Id: 71_YAMAHA_AVR.pm 10278 2015-12-28 09:51:20Z markusbloch $


volumeStraight hat Minus-Zeichen im Argument. Ich hab die Regex erweitert (Zeile 440 - [-])

Code Auswählen Erweitern


elsif(defined($a[2]) and $a[2] =~ /^[-]\d+(?:\.\d)?$/)


Sonst kommt $usage zum vorschein.
Ist mir beim Lautstärkeabsenken beim Tel-Klingeln aufgefallen.

getestet, i.O. -- Modul-Code anbei zum diffen.

Grüße
Steffen

[Markus Bloch]

Hallo Steffen,

stimmt, da hab ich nicht aufgepasst. Da mich heute noch niemand angerufen hat und der Fernseher noch nicht an war, habe ich das auch nicht bemerkt. Prinzipiell muss aber kein Minus-Zeichen davor sein (wenn man über 0 dB geht, die Nachbarn freuts jedenfalls), daher habe ich folgende Regexp gewählt:

Code Auswählen Erweitern

/^-?\d+(?:\.\d)?$/

Die Änderungen habe ich soeben eingecheckt und stehen ab morgen zur Verfügung.

Viele Grüße

Markus

[knopf_piano]

Hallo Steffen,

stimmt, da hab ich nicht aufgepasst. Da mich heute noch niemand angerufen hat und der Fernseher noch nicht an war, habe ich das auch nicht bemerkt. Prinzipiell muss aber kein Minus-Zeichen davor sein (wenn man über 0 dB geht, die Nachbarn freuts jedenfalls), daher habe ich folgende Regexp gewählt:

Code Auswählen Erweitern

/^-?\d+(?:\.\d)?$/

Die Änderungen habe ich soeben eingecheckt und stehen ab morgen zur Verfügung.

Viele Grüße

Markus

Right