From bf3f859736a45dd7825db0422bcf9832a368a0e7 Mon Sep 17 00:00:00 2001 From: luzpaz Date: Fri, 24 Feb 2023 19:57:09 +0000 Subject: [PATCH] OpenSCAD: lint OpenSCADUtils.py --- src/Mod/OpenSCAD/OpenSCADUtils.py | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/src/Mod/OpenSCAD/OpenSCADUtils.py b/src/Mod/OpenSCAD/OpenSCADUtils.py index 0acf13f1c0..dcf72f41ea 100644 --- a/src/Mod/OpenSCAD/OpenSCADUtils.py +++ b/src/Mod/OpenSCAD/OpenSCADUtils.py @@ -176,7 +176,7 @@ def callopenscad(inputfilename,outputfilename=None, outputext='csg', keepname=Fa elif transferMechanism == 2: # Use pipes instead of tempfiles return call_openscad_with_pipes(inputfilename, outputfilename, outputext, keepname) else: - raise OpenSCADError("Invalid transfer mechanism specified"); + raise OpenSCADError("Invalid transfer mechanism specified") if osfilename and os.path.isfile(osfilename): if not outputfilename: @@ -231,7 +231,7 @@ def call_openscad_with_pipes(input_filename, output_filename, output_extension, output_filename=os.path.join(dir1,'%s.%s' % \ (next(tempfilenamegen),output_extension)) with open(output_filename,"w") as outfile: - outfile.write(stdoutd); + outfile.write(stdoutd) return output_filename return None @@ -487,7 +487,7 @@ def roundrotation(rot, maxangulardistance=1e-5): for euler in itertools.permutations((0,0,angle)): eulers.append(euler) for euler in itertools.product((0,45,90,135,180,-45,-90,-135), repeat=3): - eulers.append(euler) + eulers.append(euler) for euler in eulers: r2 = FreeCAD.Rotation(*euler) if comparerotations(r1, r2) < maxangulardistance: @@ -587,9 +587,9 @@ def meshoponobjs(opname, inobjs): else: pass #neither a mesh nor a part if len(objs) > 0: - return (meshoptempfile(opname,meshes),objs) + return (meshoptempfile(opname,meshes),objs) else: - return (None,[]) + return (None,[]) def process2D_ObjectsViaOpenSCADShape(ObjList, Operation, doc): @@ -634,9 +634,9 @@ def process2D_ObjectsViaOpenSCAD(ObjList, Operation, doc=None): obj.Shape=face # Hide Children if FreeCAD.GuiUp: - for index in ObjList : - index.ViewObject.hide() - return(obj) + for index in ObjList : + index.ViewObject.hide() + return obj def process3D_ObjectsViaOpenSCADShape(ObjList, Operation, maxmeshpoints=None): @@ -661,7 +661,7 @@ def process3D_ObjectsViaOpenSCADShape(ObjList, Operation, maxmeshpoints=None): solid = Part.Solid(sh) solid = solid.removeSplitter() if solid.Volume < 0: - solid.complement() + solid.complement() return solid @@ -671,9 +671,9 @@ def process3D_ObjectsViaOpenSCAD(doc,ObjList, Operation): obj = doc.addObject('Part::Feature',Operation) #non-parametric object obj.Shape=solid#.removeSplitter() if FreeCAD.GuiUp: - for index in ObjList: - index.ViewObject.hide() - return(obj) + for index in ObjList: + index.ViewObject.hide() + return obj def process_ObjectsViaOpenSCADShape(doc, children, name, maxmeshpoints=None):