Code Linter (C++)

Fixed stucked cltool -reset #2

# to view logs

Summary
Jobs
- cpp-linter
Run details
- Usage
- Workflow file

Triggered via push May 16, 2024 11:41

honzov

reconnect-on-failure1.11.2

Status Success

Total duration 38s

Artifacts –

cpp-linter.yml

on: push

Annotations

10 warnings

src/cltool_main.cpp:39:10 [modernize-deprecated-headers]: src/cltool_main.cpp#L39

inclusion of deprecated C++ header 'signal.h'; consider using 'csignal' instead

src/cltool_main.cpp:43:13 [cppcoreguidelines-avoid-non-const-global-variables]: src/cltool_main.cpp#L43

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

src/cltool_main.cpp:123:60 [readability-implicit-bool-conversion]: src/cltool_main.cpp#L123

implicit conversion 'uint32_t' (aka 'unsigned int') -> bool

src/cltool_main.cpp:178:13 [modernize-use-trailing-return-type]: src/cltool_main.cpp#L178

use a trailing return type for this function

src/cltool_main.cpp:178:13 [readability-function-cognitive-complexity]: src/cltool_main.cpp#L178

function 'cltool_setupCommunications' has cognitive complexity of 27 (threshold 25)

src/cltool_main.cpp:182:6 [readability-container-size-empty]: src/cltool_main.cpp#L182

the 'empty' method should be used to check for emptiness instead of 'size'

src/cltool_main.cpp:192:6 [readability-implicit-bool-conversion]: src/cltool_main.cpp#L192

implicit conversion 'eDataIDs' (aka 'unsigned int') -> bool

src/cltool_main.cpp:197:6 [readability-braces-around-statements]: src/cltool_main.cpp#L197

statement should be inside braces

src/cltool_main.cpp:206:52 [cppcoreguidelines-pro-type-cstyle-cast]: src/cltool_main.cpp#L206

do not use C-style cast to convert between unrelated types

src/cltool_main.cpp:220:51 [cppcoreguidelines-pro-type-cstyle-cast]: src/cltool_main.cpp#L220

do not use C-style cast to convert between unrelated types