BIM: Fix tabstop order for Project Manager dialog #1410
Annotations
7 errors and 43 warnings
Codespell
Process completed with exit code 1.
|
Check for trailing whitespaces
Process completed with exit code 1.
|
Black (Python)
Process completed with exit code 1.
|
Pylint:
src/Mod/Draft/importDXF.py#L1495
Possibly using variable 'z' before assignment (possibly-used-before-assignment)
|
Pylint:
src/Mod/Draft/importDXF.py#L3521
Possibly using variable 'inl' before assignment (possibly-used-before-assignment)
|
Pylint
Process completed with exit code 1.
|
Cpplint
Process completed with exit code 1.
|
Codespell:
src/Mod/Draft/importDXF.py#L2157
LightYears ==> light years
|
Check for trailing whitespaces:
src/Mod/Draft/importDXF.py#L1945
<-- trailing whitespace
|
Black (Python):
src/Mod/PartDesign/PartDesignTests/TestPad.py#L1
would reformat src/Mod/PartDesign/PartDesignTests/TestPad.py
|
Black (Python):
src/Mod/Draft/draftviewproviders/view_layer.py#L1
would reformat src/Mod/Draft/draftviewproviders/view_layer.py
|
Black (Python):
src/Mod/Sketcher/SketcherTests/TestSketcherSolver.py#L1
would reformat src/Mod/Sketcher/SketcherTests/TestSketcherSolver.py
|
Black (Python):
src/Mod/Draft/importDXF.py#L1
would reformat src/Mod/Draft/importDXF.py
|
Pylint:
src/Mod/Draft/draftviewproviders/view_layer.py#L32
Use 'from pivy import coin' instead (consider-using-from-import)
|
Pylint:
src/Mod/Draft/draftviewproviders/view_layer.py#L33
Use 'from PySide import QtCore' instead (consider-using-from-import)
|
Pylint:
src/Mod/Draft/draftviewproviders/view_layer.py#L34
Use 'from PySide import QtGui' instead (consider-using-from-import)
|
Pylint:
src/Mod/Draft/draftviewproviders/view_layer.py#L187
Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
|
Pylint:
src/Mod/Draft/draftviewproviders/view_layer.py#L197
Unused argument 'vobj' (unused-argument)
|
Pylint:
src/Mod/Draft/draftviewproviders/view_layer.py#L213
Unused argument 'state' (unused-argument)
|
Pylint:
src/Mod/Draft/draftviewproviders/view_layer.py#L232
Unnecessary "elif" after "return", remove the leading "el" from "elif" (no-else-return)
|
Pylint:
src/Mod/Draft/draftviewproviders/view_layer.py#L335
Import outside toplevel (draftmake.make_layer.get_layer_container) (import-outside-toplevel)
|
Pylint:
src/Mod/Draft/draftviewproviders/view_layer.py#L449
Unnecessary "elif" after "continue", remove the leading "el" from "elif" (no-else-continue)
|
Pylint:
src/Mod/Draft/draftviewproviders/view_layer.py#L438
Too many nested blocks (6/5) (too-many-nested-blocks)
|
Cpplint:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L73
Do not use namespace using-directives. Use using-declarations instead. [build/namespaces] [5]
|
Cpplint:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L85
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
Cpplint:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L91
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
Cpplint:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L103
Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
|
Cpplint:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L109
An else should appear on the same line as the preceding } [whitespace/newline] [4]
|
Cpplint:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L109
If an else has a brace on one side, it should have it on both [readability/braces] [5]
|
Cpplint:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L116
Using C-style cast. Use reinterpret_cast<Layer*>(...) instead [readability/casting] [4]
|
Cpplint:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L123
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
Cpplint:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L138
{ should almost always be at the end of the previous line [whitespace/braces] [4]
|
Cpplint:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L150
An else should appear on the same line as the preceding } [whitespace/newline] [4]
|
Clazy:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L607
field 'GroupContents' will be initialized after field 'DraftLayerView' [-Wreorder-ctor]
|
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]
|
Clang-tidy:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L116
do not use C-style cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-cstyle-cast]
|
Clang-tidy:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L209
0.00000001 is a magic number; consider replacing it with a named constant [cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers]
|
Clang-tidy:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L430
do not call c-style vararg functions [cppcoreguidelines-pro-type-vararg]
|
Clang-tidy:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L556
do not call c-style vararg functions [cppcoreguidelines-pro-type-vararg]
|
Clang-tidy:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L607
field 'GroupContents' will be initialized after field 'DraftLayerView' [clang-diagnostic-reorder-ctor]
|
Clang-tidy:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L610
do not use C-style cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-cstyle-cast]
|
Clang-tidy:
src/Mod/Import/App/dxf/ImpExpDxf.cpp#L610
do not use C-style cast to downcast from a base to a derived class [cppcoreguidelines-pro-type-cstyle-cast]
|
Loading