Skip to content
Shirakami-CI

NOLINT #3878

Sign in to view logs

NOLINT

NOLINT #3878

Triggered via push January 16, 2025 09:47
@ban-nobuhiroban-nobuhiro
pushed 41fece7
no-nolint
Status Failure
Total duration 2m 34s
Artifacts –

ci-build.yml

on: push
Analysis
2m 23s
Analysis
Matrix: Build
Fit to window
Zoom out
Zoom in

Annotations

1 error and 19 warnings
Analysis
cppcoreguidelines-avoid-goto,hicpp-avoid-goto: src/concurrency_control/interface/long_tx/termination.cpp#L767
avoid using 'goto' for flow control
misc-misplaced-const: src/concurrency_control/interface/long_tx/termination.cpp#L900
'token' declared with a const-qualified type alias; results in the type being 'void *const' instead of 'const void *'
performance-unnecessary-value-param: src/concurrency_control/interface/tx_begin.cpp#L39
the parameter 'options' is copied for each invocation but only used as a const reference; consider making it a const reference
performance-unnecessary-value-param: src/concurrency_control/interface/tx_begin.cpp#L111
the parameter 'options' is copied for each invocation but only used as a const reference; consider making it a const reference
bugprone-narrowing-conversions,cppcoreguidelines-narrowing-conversions: src/concurrency_control/interface/start_up.cpp#L83
narrowing conversion from 'long' to signed type 'int' is implementation-defined
cppcoreguidelines-pro-type-vararg,hicpp-vararg: src/concurrency_control/interface/start_up.cpp#L83
do not call c-style vararg functions
google-readability-casting: src/concurrency_control/interface/start_up.cpp#L94
C-style casts are discouraged; use constructor call syntax
performance-unnecessary-value-param: src/concurrency_control/interface/start_up.cpp#L234
the parameter 'options' is copied for each invocation but only used as a const reference; consider making it a const reference
bugprone-implicit-widening-of-multiplication-result: src/concurrency_control/interface/helper.cpp#L42
performing an implicit widening conversion to type 'unsigned long' of a multiplication performed in type 'int'
cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers: src/concurrency_control/interface/helper.cpp#L42
30 is a magic number; consider replacing it with a named constant
cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers: src/concurrency_control/interface/helper.cpp#L42
1024 is a magic number; consider replacing it with a named constant
fuchsia-default-arguments-declarations: src/concurrency_control/interface/helper.cpp#L109
declaring a parameter with a default argument is disallowed
cppcoreguidelines-owning-memory: src/concurrency_control/garbage.cpp#L126
deleting a pointer through a type that is not marked 'gsl::owner<>'; consider using a smart pointer instead
cppcoreguidelines-pro-type-reinterpret-cast: src/concurrency_control/garbage.cpp#L277
do not use reinterpret_cast
hicpp-use-auto,modernize-use-auto: src/concurrency_control/garbage.cpp#L311
use auto when initializing with a cast to avoid duplicating the type name
cppcoreguidelines-pro-type-reinterpret-cast: src/concurrency_control/garbage.cpp#L311
do not use reinterpret_cast
cppcoreguidelines-owning-memory: src/concurrency_control/garbage.cpp#L355
deleting a pointer through a type that is not marked 'gsl::owner<>'; consider using a smart pointer instead
cppcoreguidelines-owning-memory: src/concurrency_control/garbage.cpp#L370
deleting a pointer through a type that is not marked 'gsl::owner<>'; consider using a smart pointer instead
bugprone-narrowing-conversions,cppcoreguidelines-narrowing-conversions: src/concurrency_control/garbage.cpp#L378
narrowing conversion from 'std::size_t' (aka 'unsigned long') to signed type '__gnu_cxx::__normal_iterator<std::pair<shirakami::Record *, unsigned long> *, std::vector<std::pair<shirakami::Record *, unsigned long>>>::difference_type' (aka 'long') is implementation-defined