Fix undefined behavior compile error (#346) #1060

error: try not to call a closure in the expression where it is declared --> src/pwm.rs:320:13 | 320 | $enable_break_timer(&tim); | ^^^^^^^^^^^^^^^^^^^^^^^^^ ... 1368 | pwm_timer_with_break!(tim17, TIM17, u16, pclk2, [Tim17Ch1], [PwmChannel]); | ------------------------------------------------------------------------- in this macro invocation | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure_call = note: this error originates in the macro `pwm_timer_private` which comes from the expansion of the macro `pwm_timer_with_break` (in Nightly builds, run with -Z macro-backtrace for more info)

try not to call a closure in the expression where it is declared: src/pwm.rs#L320

error: try not to call a closure in the expression where it is declared --> src/pwm.rs:320:13 | 320 | $enable_break_timer(&tim); | ^^^^^^^^^^^^^^^^^^^^^^^^^ ... 1339 | pwm_timer_with_break!(tim16, TIM16, u16, pclk2, [Tim16Ch1], [PwmChannel]); | ------------------------------------------------------------------------- in this macro invocation | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure_call = note: this error originates in the macro `pwm_timer_private` which comes from the expansion of the macro `pwm_timer_with_break` (in Nightly builds, run with -Z macro-backtrace for more info)

try not to call a closure in the expression where it is declared: src/pwm.rs#L320

error: try not to call a closure in the expression where it is declared --> src/pwm.rs:320:13 | 320 | $enable_break_timer(&tim); | ^^^^^^^^^^^^^^^^^^^^^^^^^ ... 1285 | / pwm_timer_with_break!( 1286 | | tim15, 1287 | | TIM15, 1288 | | u16, ... | 1291 | | [PwmChannel, PwmChannel] 1292 | | ); | |_- in this macro invocation | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure_call = note: this error originates in the macro `pwm_timer_private` which comes from the expansion of the macro `pwm_timer_with_break` (in Nightly builds, run with -Z macro-backtrace for more info)

try not to call a closure in the expression where it is declared: src/pwm.rs#L320

error: try not to call a closure in the expression where it is declared --> src/pwm.rs:320:13 | 320 | $enable_break_timer(&tim); | ^^^^^^^^^^^^^^^^^^^^^^^^^ ... 1162 | tim8!(); | ------- in this macro invocation | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure_call = note: this error originates in the macro `pwm_timer_private` which comes from the expansion of the macro `tim8` (in Nightly builds, run with -Z macro-backtrace for more info)

try not to call a closure in the expression where it is declared: src/pwm.rs#L320

error: try not to call a closure in the expression where it is declared --> src/pwm.rs:320:13 | 320 | $enable_break_timer(&tim); | ^^^^^^^^^^^^^^^^^^^^^^^^^ ... 1030 | tim4_common!(); | -------------- in this macro invocation | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure_call = note: this error originates in the macro `pwm_timer_private` which comes from the expansion of the macro `tim4_common` (in Nightly builds, run with -Z macro-backtrace for more info)

try not to call a closure in the expression where it is declared: src/pwm.rs#L320

error: try not to call a closure in the expression where it is declared --> src/pwm.rs:320:13 | 320 | $enable_break_timer(&tim); | ^^^^^^^^^^^^^^^^^^^^^^^^^ ... 910 | tim3_common!(); | -------------- in this macro invocation | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure_call = note: this error originates in the macro `pwm_timer_private` which comes from the expansion of the macro `tim3_common` (in Nightly builds, run with -Z macro-backtrace for more info)

try not to call a closure in the expression where it is declared: src/pwm.rs#L320

error: try not to call a closure in the expression where it is declared --> src/pwm.rs:320:13 | 320 | $enable_break_timer(&tim); | ^^^^^^^^^^^^^^^^^^^^^^^^^ ... 716 | / pwm_timer_basic!( 717 | | tim2, 718 | | TIM2, 719 | | u32, ... | 722 | | [PwmChannel, PwmChannel, PwmChannel, PwmChannel] 723 | | ); | |_- in this macro invocation | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure_call = note: this error originates in the macro `pwm_timer_private` which comes from the expansion of the macro `pwm_timer_basic` (in Nightly builds, run with -Z macro-backtrace for more info)

try not to call a closure in the expression where it is declared: src/pwm.rs#L320

error: try not to call a closure in the expression where it is declared --> src/pwm.rs:320:13 | 320 | $enable_break_timer(&tim); | ^^^^^^^^^^^^^^^^^^^^^^^^^ ... 695 | tim1_common!(); | -------------- in this macro invocation | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure_call = note: `-D clippy::redundant-closure-call` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::redundant_closure_call)]` = note: this error originates in the macro `pwm_timer_private` which comes from the expansion of the macro `tim1_common` (in Nightly builds, run with -Z macro-backtrace for more info)

unneeded sub `cfg` when there is only one condition: src/timer/interrupts.rs#L87

error: unneeded sub `cfg` when there is only one condition --> src/timer/interrupts.rs:87:7 | 87 | #[cfg(all(feature = "svd-f303"))] | ^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `feature = "svd-f303"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#non_minimal_cfg

unneeded sub `cfg` when there is only one condition: src/pwm.rs#L1099

error: unneeded sub `cfg` when there is only one condition --> src/pwm.rs:1099:7 | 1099 | #[cfg(any(feature = "stm32f373"))] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `feature = "stm32f373"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#non_minimal_cfg = note: `-D clippy::non-minimal-cfg` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::non_minimal_cfg)]`

Clippy

Clippy had exited with the 101 exit code

Link Checker

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Link Checker

The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Markdown Lint

Rustfmt

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions-rs/toolchain@v1, actions-rs/cargo@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Rustfmt

Cargo Deny

Check Codegen

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions-rs/toolchain@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Check Codegen

Check minimal feature set

Clippy

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions-rs/toolchain@v1, actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Clippy