Skip to content
lint

refactor: replace boost::json::error_code with boost::system::error_c… #1715

Sign in to view logs

refactor: replace boost::json::error_code with boost::system::error_c…

refactor: replace boost::json::error_code with boost::system::error_c… #1715

Triggered via push May 16, 2024 21:42
@cwaldren-ldcwaldren-ld
pushed 59f8beb
main
Status Success
Total duration 2m 38s
Artifacts –

cpp-linter.yml

on: push
cpp-linter
2m 26s
cpp-linter
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
/libs/client-sdk/src/data_sources/data_source_event_handler.cpp:46:54 [bugprone-unchecked-optional-access]: libs/client-sdk/src/data_sources/data_source_event_handler.cpp#L46
unchecked access to optional value
/libs/client-sdk/src/data_sources/data_source_event_handler.cpp:75:5 [modernize-pass-by-value]: libs/client-sdk/src/data_sources/data_source_event_handler.cpp#L75
pass by value and use std::move
/libs/client-sdk/src/data_sources/data_source_event_handler.cpp:85:5 [bugprone-easily-swappable-parameters]: libs/client-sdk/src/data_sources/data_source_event_handler.cpp#L85
2 adjacent parameters of 'HandleMessage' of similar type ('const std::string &') are easily swapped by mistake
/libs/client-sdk/src/flag_manager/flag_persistence.cpp:17:26 [cppcoreguidelines-pro-type-reinterpret-cast]: libs/client-sdk/src/flag_manager/flag_persistence.cpp#L17
do not use reinterpret_cast
/libs/client-sdk/src/flag_manager/flag_persistence.cpp:32:34 [modernize-pass-by-value]: libs/client-sdk/src/flag_manager/flag_persistence.cpp#L32
pass by value and use std::move
/libs/client-sdk/src/flag_manager/flag_persistence.cpp:38:21 [performance-unnecessary-value-param]: libs/client-sdk/src/flag_manager/flag_persistence.cpp#L38
parameter 'time_stamper' is passed by value and only copied once; consider moving it to avoid unnecessary copies
/libs/client-sdk/src/flag_manager/flag_persistence.cpp:109:24 [readability-identifier-length]: libs/client-sdk/src/flag_manager/flag_persistence.cpp#L109
variable name 'v' is too short, expected at least 3 characters
/libs/client-sdk/src/flag_manager/flag_persistence.cpp:129:60 [performance-move-const-arg]: libs/client-sdk/src/flag_manager/flag_persistence.cpp#L129
passing result of std::move() as a const reference argument; no move will actually happen
/libs/client-sdk/src/flag_manager/flag_persistence.cpp:133:12 [modernize-return-braced-init-list]: libs/client-sdk/src/flag_manager/flag_persistence.cpp#L133
avoid repeating the return type from the declaration; use a braced initializer list instead
/libs/client-sdk/tests/context_index_test.cpp:27:45 [cppcoreguidelines-avoid-magic-numbers: libs/client-sdk/tests/context_index_test.cpp#L27
100 is a magic number; consider replacing it with a named constant