PD: LGTM - don't catch BaseException
This commit is contained in:
@@ -153,7 +153,7 @@ class Hole():
|
||||
secondLineIndex = i
|
||||
if (firstLineIndex > -1) and (secondLineIndex > -1):
|
||||
break
|
||||
except:
|
||||
except Exception:
|
||||
# Unknown curvetype GeomAbs_OtherCurve
|
||||
continue
|
||||
axis.References = [(support, elementList[0]), (support, "Edge" + str(firstLineIndex+1)), (support, "Edge" + str(secondLineIndex+1))]
|
||||
|
||||
@@ -61,7 +61,7 @@ class TaskHole:
|
||||
body.removeObject(sketch)
|
||||
try:
|
||||
document.removeObject(sketch.Name)
|
||||
except:
|
||||
except Exception:
|
||||
pass # This always throws an exception: "Sketch support has been deleted" from SketchObject::execute()
|
||||
body.removeObject(plane)
|
||||
document.removeObject(plane.Name)
|
||||
|
||||
Reference in New Issue
Block a user