[pre-commit.ci] auto fixes from pre-commit.com hooks #209
Annotations
1 error and 28 warnings
|
Cpplint
Process completed with exit code 1.
|
|
Cpplint:
src/Mod/Measure/App/MeasureArea.cpp#L32
Do not use namespace using-directives. Use using-declarations instead. [build/namespaces] [5]
|
|
Cpplint:
src/Mod/Measure/App/MeasureArea.cpp#L38
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
|
Cpplint:
src/Mod/Measure/App/MeasureArea.cpp#L58
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
|
Cpplint:
src/Mod/Measure/App/MeasureArea.cpp#L59
Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
|
|
Cpplint:
src/Mod/Measure/App/MeasureArea.cpp#L71
Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
|
|
Cpplint:
src/Mod/Measure/App/MeasureArea.cpp#L80
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
|
Cpplint:
src/Mod/Measure/App/MeasureArea.cpp#L98
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
|
Cpplint:
src/Mod/Measure/App/MeasureArea.cpp#L124
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
|
Cpplint:
src/Mod/Measure/App/MeasureArea.cpp#L139
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
|
Cpplint:
src/Mod/Measure/App/MeasureArea.cpp#L161
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
|
Clazy:
src/Mod/Measure/App/MeasureArea.cpp#L110
unused variable 'ret' [-Wunused-variable]
|
|
Clazy:
build/_deps/fmt-src/include/fmt/core.h#L971
fmt::detail::iterator_buffer has dtor but not copy-ctor, copy-assignment [-Wclazy-rule-of-three]
|
|
Clazy:
build/_deps/fmt-src/include/fmt/core.h#L1002
fmt::detail::iterator_buffer has dtor but not copy-ctor, copy-assignment [-Wclazy-rule-of-three]
|
|
Clazy:
build/_deps/fmt-src/include/fmt/core.h#L2095
Returning a void expression [-Wclazy-returning-void-expression]
|
|
Clazy:
build/_deps/fmt-src/include/fmt/core.h#L2197
fmt::detail::specs_setter has copy-ctor but not dtor, copy-assignment [-Wclazy-rule-of-three]
|
|
Clazy:
build/_deps/fmt-src/include/fmt/core.h#L2233
fmt::detail::dynamic_specs_handler has copy-ctor but not dtor, copy-assignment [-Wclazy-rule-of-three]
|
|
Clazy:
build/_deps/fmt-src/include/fmt/core.h#L3126
Pass small and trivially-copyable type by value (const signed char &) [-Wclazy-function-args-by-value]
|
|
Clazy:
build/_deps/fmt-src/include/fmt/core.h#L3127
Pass small and trivially-copyable type by value (const unsigned char &) [-Wclazy-function-args-by-value]
|
|
Clazy:
build/_deps/fmt-src/include/fmt/core.h#L3128
Pass small and trivially-copyable type by value (const short &) [-Wclazy-function-args-by-value]
|
|
Clazy:
build/_deps/fmt-src/include/fmt/core.h#L3129
Pass small and trivially-copyable type by value (const unsigned short &) [-Wclazy-function-args-by-value]
|
|
Clang-tidy:
src/Mod/Measure/App/MeasureArea.cpp#L64
replace loop by 'std::all_of()' [readability-use-anyofallof]
|
|
Clang-tidy:
src/Mod/Measure/App/MeasureArea.cpp#L64
loop variable is copied but only used as const reference; consider making it a const reference [performance-for-range-copy]
|
|
Clang-tidy:
src/Mod/Measure/App/MeasureArea.cpp#L86
loop variable is copied but only used as const reference; consider making it a const reference [performance-for-range-copy]
|
|
Clang-tidy:
src/Mod/Measure/App/MeasureArea.cpp#L110
use auto when initializing with new to avoid duplicating the type name [modernize-use-auto]
|
|
Clang-tidy:
src/Mod/Measure/App/MeasureArea.cpp#L110
Value stored to 'ret' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
|
|
Clang-tidy
note: Value stored to 'ret' during its initialization is never read
|
|
Clang-tidy:
src/Mod/Measure/App/MeasureArea.cpp#L110
unused variable 'ret' [clang-diagnostic-unused-variable]
|
|
Clang-tidy:
src/Mod/Measure/App/MeasureArea.cpp#L144
avoid repeating the return type from the declaration; use a braced initializer list instead [modernize-return-braced-init-list]
|
Loading