kaidokert / rm32 / 25193999729

62%

main: 63%

Build Type

Pull #22

github

Committed by kaidokert

Commit Message

Tighten visibility to pub(crate), delete dead code surfaced by compiler

Change pub to pub(crate) on all struct fields, free functions, and
types not accessed from outside the rm32 crate. This surfaced 12
dead items that were hidden by pub suppressing dead_code warnings:

Deleted structs:
- InputState (state.rs) — shadowed by the real InputState in control/input.rs
- TelemetryState — only field (send_telemetry) was never read from struct

Deleted fields:
- BemfState::advance — written but never read
- MainState::telemetry — TelemetryState field never read

Deleted functions:
- throttle_to_setpoint, clamp_startup_duty — extracted but never wired up
- CurrentFilter::reset, EwmaPow2::value — never called

Deleted constants:
- SINE_SLOW_STEP_MIN, SINE_DUTY_OFFSET — only used by deleted functions

Pull Request Pull Request #22: Tighten visibility to pub(crate), delete 12 dead items surfaced by compiler

Coverage Stats

26 of 28 new or added lines in 7 files covered. (92.86%)

60 existing lines in 1 file now uncovered.

3201 of 5203 relevant lines covered (61.52%)

605.52 hits per line