jandelgado / jled / 30822779135

97%

master: 97%

Build Type

Pull #183

github

Committed by jandelgado

Commit Message

fix: Esp32Hal::SetLowActive() drop the buggy duty=0 restore

Always carry the current duty/hpoint through unchanged, instead of
special-casing duty==0 to force full-on. Matches PicoHal's simpler
behavior. Wrap in InvertableHal for a software fallback if the native
LEDC invert misbehaves at some duty value.

Pull Request Pull Request #183: Move signal inversion to HAL

Coverage Stats

187 of 202 branches covered (92.57%)

Branch coverage included in aggregate %.

40 of 40 new or added lines in 4 files covered. (100.0%)

9 existing lines in 1 file now uncovered.

611 of 623 relevant lines covered (98.07%)

754.55 hits per line