perf: apply clang-tidy performance checks #6605
Triggered via pull request
October 8, 2023 14:15
scarf005
Status
Failure
Total duration
1h 44m 56s
Artifacts
–
clang-tidy.yml
on: pull_request
skip-duplicates
9s
build
1h 44m
Annotations
10 errors
|
build:
src/character_functions.cpp#L99
'liquid' used after it was moved [bugprone-use-after-move,-warnings-as-errors]
|
|
build:
src/inventory.cpp#L1411
'newit' used after it was moved [bugprone-use-after-move,-warnings-as-errors]
|
|
build:
src/inventory.cpp#L1443
'newit' used after it was moved [bugprone-use-after-move,-warnings-as-errors]
|
|
build:
src/inventory.cpp#L1472
'newit' used after it was moved [bugprone-use-after-move,-warnings-as-errors]
|
|
build:
src/submap.cpp#L63
Prefer constructing 'point' from named constant 'point_zero' rather than explicit integer arguments. [cata-use-named-point-constants,-warnings-as-errors]
|
|
build:
src/submap.cpp#L64
Prefer constructing 'point' from named constant 'point_south' rather than explicit integer arguments. [cata-use-named-point-constants,-warnings-as-errors]
|
|
build:
src/submap.cpp#L77
Prefer constructing 'point' from named constant 'point_east' rather than explicit integer arguments. [cata-use-named-point-constants,-warnings-as-errors]
|
|
build:
src/submap.cpp#L78
Prefer constructing 'point' from named constant 'point_south_east' rather than explicit integer arguments. [cata-use-named-point-constants,-warnings-as-errors]
|
|
build:
src/submap.cpp#L234
use '= default' to define a trivial destructor [modernize-use-equals-default,-warnings-as-errors]
|
|
build:
src/submap.h#L221
function 'submap::swap' has a definition with different parameter names [readability-inconsistent-declaration-parameter-name,-warnings-as-errors]
|