Draft: move more functions to draftgeoutils.faces

src/Mod/Draft/DraftGeomUtils.py

@@ -380,27 +380,7 @@ from draftgeoutils.wires import tessellateProjection
 from draftgeoutils.wires import rebaseWire
 
 
-def removeSplitter(shape):
-    """an alternative, shared edge-based version of Part.removeSplitter. Returns a
-    face or None if the operation failed"""
-    lut = {}
-    for f in shape.Faces:
-        for e in f.Edges:
-            h = e.hashCode()
-            if h in lut:
-                lut[h].append(e)
-            else:
-                lut[h] = [e]
-    edges = [e[0] for e in lut.values() if len(e) == 1]
-    try:
-        face = Part.Face(Part.Wire(edges))
-    except:
-        # operation failed
-        return None
-    else:
-        if face.isValid():
-            return face
-    return None
+from draftgeoutils.faces import removeSplitter
 
 
 # circle functions *********************************************************

src/Mod/Draft/draftgeoutils/faces.py

@@ -229,3 +229,32 @@ def cleanFaces(shape):
     if shape.isClosed():
         fshape = Part.makeSolid(fshape)
     return fshape
+
+
+def removeSplitter(shape):
+    """Return a face from removing the splitter in a list of faces.
+
+    This is an alternative, shared edge-based version of Part.removeSplitter.
+    Returns a face, or `None` if the operation failed.
+    """
+    lookup = dict()
+    for f in shape.Faces:
+        for e in f.Edges:
+            h = e.hashCode()
+            if h in lookup:
+                lookup[h].append(e)
+            else:
+                lookup[h] = [e]
+
+    edges = [e[0] for e in lookup.values() if len(e) == 1]
+
+    try:
+        face = Part.Face(Part.Wire(edges))
+    except:
+        # operation failed
+        return None
+    else:
+        if face.isValid():
+            return face
+
+    return None