Closed elektron-bbs closed 2 years ago
Merging #1109 (b668275) into master (84cb986) will increase coverage by
0.01%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #1109 +/- ##
==========================================
+ Coverage 64.67% 64.68% +0.01%
==========================================
Files 132 132
Lines 9643 9643
Branches 1530 1530
==========================================
+ Hits 6237 6238 +1
+ Misses 2213 2212 -1
Partials 1193 1193
Flag | Coverage Δ | |
---|---|---|
fhem | 56.56% <100.00%> (ø) |
|
modules | 64.68% <100.00%> (+0.01%) |
:arrow_up: |
perl | 90.37% <ø> (+0.20%) |
:arrow_up: |
unittests | 64.68% <100.00%> (+0.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
FHEM/14_SD_WS.pm | 65.47% <100.00%> (ø) |
|
FHEM/lib/SD_Protocols.pm | 79.10% <0.00%> (+0.10%) |
:arrow_up: |
:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more
Was ich nicht verstehe ist, wie sich diese Änderung auf Zeile 1300-1302 auswirken soll. Ist die Logangabe aktuell?
Die Logausgabe ist aktuell. Irgendwie schafft es da doch ab und an eine Nachricht durch die CRC-Prüfung zu kommen. Da im prematch für die BCD-kodierten Teile jetzt nur noch Ziffern von 0 bis 9 zugelassen sind, dürfte das dann zu keinen Warnungen mehr führen.
Die Logausgabe ist aktuell. Irgendwie schafft es da doch ab und an eine Nachricht durch die CRC-Prüfung zu kommen. Da im prematch für die BCD-kodierten Teile jetzt nur noch Ziffern von 0 bis 9 zugelassen sind, dürfte das dann zu keinen Warnungen mehr führen.
Sehr komisch, denn Zeile 1300 und folgend https://github.com/RFD-FHEM/RFFHEM/blob/b66827502418db2b99cffe7e2fc0a1ceab30d894/FHEM/14_SD_WS.pm#L1300
werden doch nur für Protokoll 37 aufgerufen und ein multiplizieren findet hier nicht statt. Das ist doch irgendwie faul oder?
Ach, sorry, an die Zeilennummern hatte ich nicht gedacht. Die Perl-Warnungen beziehen sich in der aktuellen Datei auf die Zeilen 1264-1266:
hum => sub {my ($rawData,undef) = @_; return substr($rawData,32,2) * 1;},
brightness => sub {my ($rawData,undef) = @_; return substr($rawData,34,6) * 0.001;},
uv => sub {my ($rawData,undef) = @_; return substr($rawData,40,3) * 0.1;},
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
[x] Bugfix (please link issue)
[ ] Feature enhancement
[ ] Documentation update
[ ] Unittest enhancement
[ ] other
What is the current behavior? (You can also link to an open issue here, if this describes the current behavior) Perl Warnings:
What is the new behavior (if this is a feature change)? no warnings
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) no
Other information: