kaidokert / rm32

61%

main: 63%

LAST BUILD ON BRANCH pr18_1

branch: pr18_1

CHANGE BRANCH

x

Reset

Sync Branches

• pr18_1
• main
• pr1
• pr10_1
• pr11_1
• pr12_1
• pr13_1
• pr14_1
• pr15_1
• pr16_1
• pr17_1
• pr19_1
• pr20_1
• pr21_1
• pr22_1
• pr23_1
• pr24_1
• pr25_1
• pr26_1
• pr27_1
• pr28_1
• pr29_1
• pr2_1
• pr2_2
• pr2_3
• pr2_4
• pr2_5
• pr2_6
• pr2_7
• pr2_ref
• pr30_1
• pr31_1
• pr32_1
• pr33_1
• pr34_1
• pr3_1
• pr5_1
• pr7_1
• pr8_1
• pr_codecov1
• pr_codecov2
• system-tick
• tmps

Build # 25197482230

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