1. Replace hard coded workbech icons with image files.
A "template" commit: https://github.com/FreeCAD/FreeCAD_sf_m ... 2a8fc91b57
Files that need to be looked at:
src/Mod/ReverseEngineering/InitGui.py /
src/Mod/Draft/InitGui.py /
src/Mod/TemplatePyMod/InitGui.py /
src/Mod/JtReader/InitGui.py /
src/Mod/Complete/InitGui.py /
src/Mod/Points/InitGui.py /
src/Mod/PartDesign/InitGui.py /
src/Mod/Arch/InitGui.py /
src/Mod/Test/InitGui.py /
src/Mod/Mesh/InitGui.py /
src/Mod/Cam/InitGui.py /
src/Mod/Part/InitGui.py /
src/Mod/OpenSCAD/InitGui.py /
src/Mod/Import/InitGui.py /
src/Mod/MeshPart/InitGui.py /
src/Mod/Assembly/InitGui.py /
src/Mod/Image/InitGui.py /
src/Mod/Sketcher/InitGui.py /
src/Mod/Spreadsheet/InitGui.py /
src/Mod/Robot/InitGui.py /
src/Mod/Web/InitGui.py /
src/Mod/Start/InitGui.py /
src/Mod/Drawing/InitGui.py /
src/Mod/Raytracing/InitGui.py
2. Removing imported but unused python modules. flake8 shows a few of them ( 442 to be precise), but there might be some false-positives, so it needs to be done carefully. The full list: http://www.pasteall.org/57800 generated with:
Code: Select all
fedora-lan:/home/przemo/fedora/FreeCAD/freecad
$ find . -name "*.py" | xargs -I {} flake8 --select=F401 {}[code]
3.Cleaning blank lines with white characters (1379 of them). Generated with:
Code: Select all
fedora-lan:/home/przemo/fedora/FreeCAD/freecad
$ find . -name "*.py" | xargs -I {} flake8 --select=W293
4. Cleaning assigned but never used local variables. Full list: http://www.pasteall.org/57801 generated with:
Code: Select all
fedora-lan:/home/przemo/fedora/FreeCAD/freecad
$ find . -name "*.py" | xargs -I {} flake8 --select=F841