Merge pull request #12528 from bgbsww/bgbsww-toponamingMoveEqualOperator #68
Annotations
2 errors and 23 warnings
Codespell
Process completed with exit code 1.
|
Cpplint
Process completed with exit code 1.
|
Codespell:
src/Mod/Part/App/TopoShape.h#L1558
overriden ==> overridden
|
Codespell:
tests/src/Mod/Part/App/TopoShapeMakeShape.cpp#L100
TRU ==> THROUGH, TRUE
|
Cpplint:
src/Mod/Part/App/TopoShape.cpp#L1986
Missing space around colon in range-based for loop [whitespace/forcolon] [2]
|
Clazy:
src/Mod/Part/App/TopoShape.h#L1009
'Part::TopoShape::resetElementMap' hides overloaded virtual function [-Woverloaded-virtual]
|
Clazy
note: hidden overloaded virtual function 'Data::ComplexGeoData::resetElementMap' declared here: different number of parameters (2 vs 1)
|
Clazy:
src/Mod/Part/App/TopoShape.cpp#L194
non-POD static (LogLevel) [-Wclazy-non-pod-global-static]
|
Clazy
note: expanded from macro 'FC_LOG_LEVEL_INIT'
|
Clazy
note: expanded from macro '_FC_LOG_LEVEL_INIT'
|
Clazy:
src/Mod/Part/App/TopoShape.cpp#L442
non-POD static (array) [-Wclazy-non-pod-global-static]
|
Clazy:
src/Mod/Part/App/TopoShape.h#L1009
'Part::TopoShape::resetElementMap' hides overloaded virtual function [-Woverloaded-virtual]
|
Clazy
note: hidden overloaded virtual function 'Data::ComplexGeoData::resetElementMap' declared here: different number of parameters (2 vs 1)
|
Clazy
'iterator<std::forward_iterator_tag, OSD_Parallel::UniversalIterator>' is deprecated [-Wdeprecated-declarations]
|
Clazy
note: 'iterator<std::forward_iterator_tag, OSD_Parallel::UniversalIterator>' has been explicitly marked deprecated here
|
Clang-tidy:
src/Mod/Part/App/TopoShape.cpp#L194
variable '_s_fclvl' is non-const and globally accessible, consider making it const [cppcoreguidelines-avoid-non-const-global-variables]
|
Clang-tidy
note: expanded from macro 'FC_LOG_LEVEL_INIT'
|
Clang-tidy
note: expanded from macro 'FC_LOG_INSTANCE'
|
Clang-tidy:
src/Mod/Part/App/TopoShape.cpp#L231
0.1 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers]
|
Clang-tidy:
src/Mod/Part/App/TopoShape.cpp#L231
5 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers]
|
Clang-tidy:
src/Mod/Part/App/TopoShape.cpp#L231
0.005 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers]
|
Clang-tidy:
src/Mod/Part/App/TopoShape.cpp#L237
initializer for base class 'Base::ValueError' is redundant [readability-redundant-member-init]
|
Clang-tidy:
src/Mod/Part/App/TopoShape.cpp#L254
initializer for base class 'Base::CADKernelError' is redundant [readability-redundant-member-init]
|
Clang-tidy:
src/Mod/Part/App/TopoShape.cpp#L270
variable 'classTypeId' is non-const and globally accessible, consider making it const [cppcoreguidelines-avoid-non-const-global-variables]
|
Clang-tidy
note: expanded from macro 'TYPESYSTEM_SOURCE'
|
The logs for this run have expired and are no longer available.
Loading