jandelgado / jled

97%

master: 97%

Pull #154

github

make candle effect stateless 

the old candle effect was stateful (which was against our architecture,
stating that effects should be stateless), which resulted in the largest
memory footprint of all effects (12 bytes). since we use a discriminated
union, the candle effect, having the largest memory footprint, was
determining the size of the effect buffer in the TJLed class. The
refactoring now uses just 6 bytes, and the Breathe effect with 8 bytes
is now the largest member. Overall we save 4 bytes on every JLed object.

Also the brightness lookup table was re-worked to include a gamma value
of 2.2, resulting in a better visual effect.

91 of 100 branches covered (91.0%)

Branch coverage included in aggregate %.

14 of 14 new or added lines in 2 files covered. (100.0%)

362 of 365 relevant lines covered (99.18%)

847.15 hits per line