kaidokert / rm32 / 25201063715

63%

Build Type

push

github

Committed by web-flow

Commit Message

Extract MotorState sub-trait from SharedComm (#25)

* Extract MotorState sub-trait from SharedComm

First step of SharedComm decomposition: extract the 11 motor mode
methods (motor_mode, set_motor_mode, transition, armed/running/
old_routine/stepper_sine + setters) into a dedicated MotorState trait.

SharedComm now requires MotorState as a super-trait. SharedState and
TestShared implement MotorState separately. Callers using SharedComm
as a trait bound get MotorState methods automatically; callers using
concrete types need MotorState imported.

This is the pilot for the sub-trait pattern described in
SHAREDCOMM_AUDIT.md — validates the approach before extracting
IsrTiming and MainControl.

* Override MotorState convenience setters with atomic CAS on SharedState

The trait default implementations use non-atomic load+store. SharedState
has atomic fetch_update inherent methods for set_armed, set_running,
set_old_routine, set_stepper_sine. Override the trait methods to
delegate to those, preventing race conditions when accessed through
the MotorState trait.

Coverage Stats

0 of 12 new or added lines in 1 file covered. (0.0%)

2 existing lines in 2 files now uncovered.

3338 of 5479 relevant lines covered (60.92%)

624.48 hits per line