Improve comments, add test #1194
Annotations
2 errors and 41 warnings
Black (Python)
Process completed with exit code 1.
|
Cpplint
Process completed with exit code 1.
|
Black (Python):
src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py#L1
would reformat src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py
|
Pylint:
src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py#L579
Line too long (105/100) (line-too-long)
|
Pylint:
src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py#L583
Line too long (101/100) (line-too-long)
|
Pylint:
src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py#L661
Line too long (105/100) (line-too-long)
|
Pylint:
src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py#L807
Line too long (105/100) (line-too-long)
|
Pylint:
src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py#L836
Line too long (108/100) (line-too-long)
|
Pylint:
src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py#L1883
Line too long (118/100) (line-too-long)
|
Pylint:
src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py#L1886
Line too long (117/100) (line-too-long)
|
Pylint:
src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py#L1889
Line too long (118/100) (line-too-long)
|
Pylint:
src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py#L1892
Line too long (118/100) (line-too-long)
|
Pylint:
src/Mod/PartDesign/PartDesignTests/TestTopologicalNamingProblem.py#L1904
Line too long (101/100) (line-too-long)
|
Cpplint:
src/Mod/Part/App/PartFeature.cpp#L80
Do not use namespace using-directives. Use using-declarations instead. [build/namespaces] [5]
|
Cpplint:
src/Mod/Part/App/PartFeature.cpp#L83
Missing space after , [whitespace/comma] [3]
|
Cpplint:
src/Mod/Part/App/PartFeature.cpp#L89
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
Cpplint:
src/Mod/Part/App/PartFeature.cpp#L98
Use int16/int64/etc, rather than the C type short [runtime/int] [4]
|
Cpplint:
src/Mod/Part/App/PartFeature.cpp#L99
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
Cpplint:
src/Mod/Part/App/PartFeature.cpp#L104
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
Cpplint:
src/Mod/Part/App/PartFeature.cpp#L109
Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
|
Cpplint:
src/Mod/Part/App/PartFeature.cpp#L117
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
Cpplint:
src/Mod/Part/App/PartFeature.cpp#L123
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
Cpplint:
src/Mod/Part/App/PartFeature.cpp#L124
Missing space before { [whitespace/braces] [5]
|
Clazy
NCollection_ListNode has copy-ctor, copy-assignment but not dtor [-Wclazy-rule-of-three]
|
Clazy
Pass small and trivially-copyable type by value (const class NCollection_BaseList::Iterator &) [-Wclazy-function-args-by-value]
|
Clazy
Pass small and trivially-copyable type by value (const class NCollection_BaseList::Iterator &) [-Wclazy-function-args-by-value]
|
Clazy
Pass small and trivially-copyable type by value (const class gp_XY &) [-Wclazy-function-args-by-value]
|
Clazy
Pass small and trivially-copyable type by value (const class gp_XY &) [-Wclazy-function-args-by-value]
|
Clazy
Pass small and trivially-copyable type by value (const class gp_XY &) [-Wclazy-function-args-by-value]
|
Clazy
Pass small and trivially-copyable type by value (const class gp_XY &) [-Wclazy-function-args-by-value]
|
Clazy
Pass small and trivially-copyable type by value (const class gp_XY &) [-Wclazy-function-args-by-value]
|
Clazy
Pass small and trivially-copyable type by value (const class gp_XY &) [-Wclazy-function-args-by-value]
|
Clazy
Pass small and trivially-copyable type by value (const class gp_XY &) [-Wclazy-function-args-by-value]
|
Clang-tidy:
src/Mod/Part/App/PartFeature.cpp#L110
use auto when initializing with new to avoid duplicating the type name [modernize-use-auto]
|
Clang-tidy:
src/Mod/Part/App/PartFeature.cpp#L111
statement should be inside braces [readability-braces-around-statements]
|
Clang-tidy:
src/Mod/Part/App/PartFeature.cpp#L155
function 'getExportElementName' has cognitive complexity of 189 (threshold 25) [readability-function-cognitive-complexity]
|
Clang-tidy
note: +1, including nesting penalty of 0, nesting level increased to 1
|
Clang-tidy
note: +1
|
Clang-tidy
note: +2, including nesting penalty of 1, nesting level increased to 2
|
Clang-tidy
note: +3, including nesting penalty of 2, nesting level increased to 3
|
Clang-tidy
note: +3, including nesting penalty of 2, nesting level increased to 3
|
Clang-tidy
note: +4, including nesting penalty of 3, nesting level increased to 4
|
Clang-tidy
note: +5, including nesting penalty of 4, nesting level increased to 5
|
Loading