kaidokert / rm32 / 25197482230

61%

main: 63%

Build Type

Pull #24

github

Committed by kaidokert

Commit Message

Remove &mut accessors, add specific setter methods instead

forward_mut(), measurements_mut(), timing_mut(), pid_mut() returned
mutable references to internal state — fake encapsulation that gives
callers full field access.

Replace with specific setter methods on MainState that delegate to
sub-structs: set_use_current_limit, set_use_speed_control,
set_average_interval, set_last_average_interval, set_battery_voltage,
set_actual_current.

Commutation::forward_mut() replaced with copy-in/copy-out pattern
at the CommandProcessor call site.

Pull Request Pull Request #24: Make struct fields private, add accessor methods

Coverage Stats

0 of 175 new or added lines in 5 files covered. (0.0%)

3 existing lines in 1 file now uncovered.

3338 of 5467 relevant lines covered (61.06%)

625.85 hits per line