cpp-linter

fix: Encode T330 shock intensity as 4-bit level with rolling state #915

Sign in to view logs

Triggered via push May 27, 2026 02:00

LucHeart

fix/t330-shock-intensity-encoding

Status Success

Total duration 1m 47s

Artifacts –

cpp-linter.yml

on: push

get-vars / get-vars

Annotations

10 warnings

src/radio/rmt/T330Encoder.cpp:54:60 [cppcoreguidelines-avoid-magic-numbers]: src/radio/rmt/T330Encoder.cpp#L54

7 is a magic number; consider replacing it with a named constant

src/radio/rmt/T330Encoder.cpp:53:40 [cppcoreguidelines-avoid-magic-numbers]: src/radio/rmt/T330Encoder.cpp#L53

100 is a magic number; consider replacing it with a named constant

src/radio/rmt/T330Encoder.cpp:53:34 [cppcoreguidelines-avoid-magic-numbers]: src/radio/rmt/T330Encoder.cpp#L53

15 is a magic number; consider replacing it with a named constant

src/radio/rmt/T330Encoder.cpp:41:19 [cppcoreguidelines-avoid-magic-numbers]: src/radio/rmt/T330Encoder.cpp#L41

0b10000100 is a magic number; consider replacing it with a named constant

src/radio/rmt/T330Encoder.cpp:38:17 [cppcoreguidelines-avoid-magic-numbers]: src/radio/rmt/T330Encoder.cpp#L38

0b01110010 is a magic number; consider replacing it with a named constant

src/radio/rmt/T330Encoder.cpp:35:17 [cppcoreguidelines-avoid-magic-numbers]: src/radio/rmt/T330Encoder.cpp#L35

0b01100001 is a magic number; consider replacing it with a named constant

src/radio/rmt/T330Encoder.cpp:27:32 [modernize-use-trailing-return-type]: src/radio/rmt/T330Encoder.cpp#L27

use a trailing return type for this function

src/radio/rmt/T330Encoder.cpp:24:10 [cppcoreguidelines-avoid-magic-numbers]: src/radio/rmt/T330Encoder.cpp#L24

43 is a magic number; consider replacing it with a named constant

src/radio/rmt/T330Encoder.cpp:22:34 [modernize-use-trailing-return-type]: src/radio/rmt/T330Encoder.cpp#L22

use a trailing return type for this function

src/radio/rmt/T330Encoder.cpp:20:56 [cppcoreguidelines-avoid-non-const-global-variables]: src/radio/rmt/T330Encoder.cpp#L20

variable 's_shockState' is non-const and globally accessible, consider making it const