From 54edb231bb354892c1eff91f83f5e4e241f10d0e Mon Sep 17 00:00:00 2001 From: luz paz Date: Thu, 5 May 2022 14:30:39 -0400 Subject: [PATCH] LGTM: Remove unnecessary deletion of local variables Passing a local variable to a del statement results in that variable being removed from the local namespace. When exiting a function all local variables are deleted, so it is unnecessary to explicitly delete variables in such cases. ref: https://lgtm.com/rules/1506104658325/ --- src/Mod/Draft/importDXF.py | 1 - src/Mod/Draft/importOCA.py | 2 -- 2 files changed, 3 deletions(-) diff --git a/src/Mod/Draft/importDXF.py b/src/Mod/Draft/importDXF.py index 4163174834..c6131e54f8 100644 --- a/src/Mod/Draft/importDXF.py +++ b/src/Mod/Draft/importDXF.py @@ -2721,7 +2721,6 @@ def processdxf(document, filename, getShapes=False, reComputeFlag=True): if badobjects: print("dxf: ", len(badobjects), " objects were not imported") del doc - del blockshapes def warn(dxfobject, num=None): diff --git a/src/Mod/Draft/importOCA.py b/src/Mod/Draft/importOCA.py index 0e013069d2..c95b8ae117 100644 --- a/src/Mod/Draft/importOCA.py +++ b/src/Mod/Draft/importOCA.py @@ -332,8 +332,6 @@ def parse(filename, doc): float(c[2])/255, float(c[3])/255) - del color - def decodeName(name): """Decode encoded name.