jandelgado / jled / 29275640606

98%

master: 98%

Pull #174

github

feature: TJLedGroup lifecycle events (kStart/kDone/kRepeatStart/kElementChanged)

Add GroupUpdateResult, returned by TJLedGroup::Update()/Update(t) instead of
plain bool, reporting group-level lifecycle events for both JLedGroup and
JLedRefGroup: IsStarted()/IsDone()/IsRepeatStarted()/IsElementChanged(), with
matching OnStart()/OnDone()/OnRepeatStart()/OnElementChanged() callbacks.

kStart/kDone fire once per run; kDone is latched (not diffed) since Stop()
mutates is_running_ outside of Update(). kRepeatStart fires once per
repetition, including the first, mode-independently, and composes correctly
under nesting. kElementChanged fires for Sequential groups when the active
element changes, mid-run or on wraparound to a new repetition; it never
fires in Parallel mode, for a single-element sequence, and it does not see
into nested subgroups (a nested group's own element transitions are opaque
to its parent).

Also fixes a duplicate-tick bug uncovered while adding kRepeatStart:
TJLedGroup had no tick-deduplication of its own (unlike TJLed). At a
repetition boundary, ResetLeds() puts every element back in ST_INIT, which
bypasses that element's own per-tick guard, so a second Update() call at the
same t could silently consume an extra repetition. TJLedGroup now has its
own last_update_time_ guard mirroring TJLed's.

Per-element event forwarding from individual TJLed instances through
TJLedRefGroup (with element identity) is deferred to a separate PR.

164 of 178 branches covered (92.13%)

Branch coverage included in aggregate %.

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

562 of 563 relevant lines covered (99.82%)

794.55 hits per line