-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PERL WARNING - Sammlung | dev Version #156
Comments
vermutlich nicht mehr aktuell. wenn doch dann erneut öffnen |
Hallo, BITTE das Tehma REOPEN!
trifft auf die Zeile
zu! Bezogen auf den aktuellen Commit dev-r33, HIER! |
Folgene Meldung 2 tritt auch auf.
dies trifft auf die Zeile
zu! Bezogen auf den aktuellen Commit dev-r33, HIER! |
Use of uninitialized value $returnvalues[0] evtl so:
|
fixed perl warning #156 14_SD_RSL.pm fixed perl warning
Bitte testen :) |
Leider ist immer noch eine WARNING vorhanden nachdem ich es mit deinen Zeilen getestet habe.
trifft auf Zeile
zu. Hier diese im aktuellen Commit! So wurde erst eine von beiden Warnings eliminiert. Mit der Variante von @Ralf9
@sidey79, scheint es so zu funktionierten. WARNING weg. |
Fix Warning Site: RFD-FHEM#156 (comment)
Seltsam, der Test auf @returnvalues scheint immer Erfolgreich zu sein. Vermutlich muss ich noch die Anzahl mit $returnvalues testen. Ralfs Variante testet nicht gegen returnvalues, sondern verlässt sich darauf, dass der returncode richtig gesetzt ist. |
Dann sollte man den Test erweitern. Vielleicht fällt mir auch noch was dazu ein.
|
Hier noch 2 andere Warnings welche ab und zu auftauchen.
einmal diese Zeile
bezogen auf aktuellen Commit - HIER! und dann auch diese Zeile
bezogen auf aktuellen Commit - HIER! Wenn ich richtig zähle, haben wir da bisher 3 vorhanden WARNINGS im Code. |
Ich brauche zu den Warnings den Auslöser, sonst ist es ein wenig try & error. |
Prevent accessing values outside of @bit_msg range in SIGNALduino_postDemo_WS2000 fixes #156
Ich habe die Fehler behoben. Hoffe ich habe nichts vergessen. Bitte mal testen |
fixed perl warnings #156 Added check if $returnvalues[1] is undef, which is true if the called sub returns (0, undef); undef will be passed as var to the first entry of the array Prevent accessing values outside of @bit_msg range in SIGNALduino_postDemo_WS2000 14_SD_RSL.pm fixed perl warning (cherry picked from commit 4467576) (cherry picked from commit 48cc973) (cherry picked from commit 47032ac)
Habe es test laufen und fürs erste sieht es gut aus 👍 Ich schließe diesen Faden und erst bei erneuten Fehlern werde ich hier mich zu Wort melden. |
@Ralf9 + @sidey79 Line 2957 in 13b0d28
|
Ja, die Variable ist bereits ein paar Zeilen weiter oben definiert. |
Zeile erzeugt eine WARNING
müsste es vielleicht nicht
lauten damit es keine Warning erzeugt? |
Ich glaube nicht, dass das helfen würde, denn pstr wird ja erst in der If Abfrage mit einem Wert belegt. |
Dann muss man vorher prüfen ob der Wert "füllbar" ist, denn diese Umstände kommen ja nur zum tragen, wenn $pstr=undef bleibt. |
Ein eigenes Issue wäre für diesen Fall besser geeignet. Interessant ist, welche Daten übergeben werden, damit es zu diesem Fehler kommt, denn SIGNALduino_PatternExists übergibt -1 oder $search und $search wäre identisch mit $symbol. |
@sidey79 möchtest du einen extra Issue? Unter "x" Verhältnissen trat er wieder auf bei mir. |
Hast Du die passende empfangene Nachricht dazu, dann kann ich es debuggen. |
Moin Moin, die passende Nachricht muss ich erst „fangen“ und das System auf verbose 4 laufen lassen. Das stellt leider immer ne Datenflut von Hunderten MB dar und somit kann ich von extern schwer ins Logfile schauen. Ich Probier das Glück, bin dran! |
@sidey79 hier ist der Logauszug wo die Nachricht erschien.
Ich gehe davon aus, das es diese Nachricht ist:
|
Es ist eindeutig, @sidey79 die Fehlermeldung ist reproduzierbar!
|
Ok danke. schau ich mir in Kürze an. Heute packe ich es nicht mehr, überarbeite gerade mein Testsystem |
Habe es etwas eingegrenzt:
|
removed zero key from protocol 76 to avoid perl warning fixes: #156
Hier gibt es auch noch Handlungsbedarf. Unter noch nicht erkennbaren Zusammenhängen, erscheint
Es tauchte auf, wenn ich einmal ODER set sduino_dummy raw MS;P1=393;P2=-7752;P3=-2047;P4=-3993;D=12131314141414141313131413131314141414131313141314131414131314131314131313131314131314;CP=1;SP=2;R=230;O;m1; ausführte. Leider ist es noch nicht 100% nachvollziehbar WANN! |
Hi,
ich beginne mal zu sammeln, Stand heutiges Update.
Manche kann man vielleicht fixen und so haben wir eine Übersicht.
The text was updated successfully, but these errors were encountered: