Skip to content
cpp-linter

Improve EStop Manager's atomics + set to Idle when changing pin/enabled. #913

Sign in to view logs

Improve EStop Manager's atomics + set to Idle when changing pin/enabled.

Improve EStop Manager's atomics + set to Idle when changing pin/enabled. #913

Triggered via pull request May 26, 2026 00:51
@hhvrchhvrc
synchronize #449
feat/improve-estop-atomics
Status Success
Total duration 1m 47s
Artifacts –

cpp-linter.yml

on: pull_request
get-vars  /  get-vars
19s
get-vars / get-vars
C/C++ Linter
1m 21s
C/C++ Linter
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
src/EStopManager.cpp:42:29 [cppcoreguidelines-avoid-non-const-global-variables]: src/EStopManager.cpp#L42
variable 's_estopActivatedAt' is non-const and globally accessible, consider making it const
src/EStopManager.cpp:39:19 [cppcoreguidelines-avoid-non-const-global-variables]: src/EStopManager.cpp#L39
variable 's_estopPin' is non-const and globally accessible, consider making it const
src/EStopManager.cpp:38:21 [cppcoreguidelines-avoid-non-const-global-variables]: src/EStopManager.cpp#L38
variable 's_estopTask' is non-const and globally accessible, consider making it const
src/EStopManager.cpp:36:31 [cppcoreguidelines-avoid-non-const-global-variables]: src/EStopManager.cpp#L36
variable 's_estopMutex' is non-const and globally accessible, consider making it const
include/estop/EStopManager.h:14:11 [modernize-use-trailing-return-type]: include/estop/EStopManager.h#L14
use a trailing return type for this function
include/estop/EStopManager.h:13:8 [modernize-use-trailing-return-type]: include/estop/EStopManager.h#L13
use a trailing return type for this function
include/estop/EStopManager.h:12:8 [modernize-use-trailing-return-type]: include/estop/EStopManager.h#L12
use a trailing return type for this function
include/estop/EStopManager.h:11:8 [modernize-use-trailing-return-type]: include/estop/EStopManager.h#L11
use a trailing return type for this function
include/estop/EStopManager.h:10:22 [modernize-use-trailing-return-type]: include/estop/EStopManager.h#L10
use a trailing return type for this function
include/estop/EStopManager.h:1:1 [portability-avoid-pragma-once]: include/estop/EStopManager.h#L1
avoid 'pragma once' directive; use include guards instead