jandelgado / jled / 30825557135

97%

master: 97%

Pull #184

github

perf: ArduinoHal initializes eagerly by default, lazily only where STM32 needs it

Deferring pinMode()/analogWriteResolution() to the first analogWrite() call
was only ever needed on STM32duino, which forbids touching hardware from a
global constructor. All other Arduino-compatible platforms paid for the
"already set up" flag and its branch on every analogWrite() call for no
reason.

ArduinoHal gains a kLazyInit_ template parameter (default false = eager init
in the constructor). The flag is stored via a private EBO base that is empty
when kLazyInit_ is false, so the eager path has zero extra storage and the
branch is resolved at compile time via tag dispatch. jled.h sets kLazyInit_
= true only for ARDUINO_ARCH_STM32.

187 of 202 branches covered (92.57%)

Branch coverage included in aggregate %.

13 of 13 new or added lines in 1 file covered. (100.0%)

618 of 630 relevant lines covered (98.1%)

746.3 hits per line