fix hole detection and dumb filling

src/main_window.cpp

@@ -43,10 +43,13 @@ void MainWindow::open(const QString &path) {
 	for (const VertexHandle &vh : mesh.vertices()) {
 		mesh.set_color(vh, MyMesh::Color(0, 0, 0));
 	}
-	if (glm != nullptr) mesh_viewer.removeOpenGLMesh(glm);
+
- 	for (HalfedgeStrip border : findBorders(mesh)) {
+ 	for (HalfedgeHandle border : findBorders(mesh)) {
 		setBorderColor(mesh, border, {1, 0, 0});
+		fillHoleDumb(mesh, border);
 	}
+
+	if (glm != nullptr) mesh_viewer.removeOpenGLMesh(glm);
 	glm = mesh_viewer.addOpenGLMeshFromOpenMesh(&mesh);
 	for (QAction *a : toolbar_actions) {
 		a->setEnabled(true);

src/mesh_processing.cpp

@@ -1,30 +1,54 @@
 #include "mesh_processing.h"
-#include <list>
+#include <QGenericMatrix>
+#include <unordered_set>
 
 
-std::vector<HalfedgeStrip> findBorders(MyMesh &mesh) {
+std::vector<HalfedgeHandle> findBorders(MyMesh &mesh) {
-	std::vector<HalfedgeStrip> borders(1);
+	std::vector<HalfedgeHandle> borders;
-	size_t current = 0;
+	std::set<HalfedgeHandle> ignore;
-	for (auto it = mesh.halfedges_begin(); it != mesh.halfedges_end(); ++it) {
+	for (HalfedgeHandle it : mesh.halfedges()) {
-		while (mesh.is_boundary(*it)) {
+		if (mesh.is_boundary(it)
-			borders[current].push_back(*it);
+			&& ignore.find(it) == ignore.end()) {
-			++it;
+			borders.push_back(it);
+			ignore.insert(it);
+			for (HalfedgeHandle it2 : HalfedgeLoopRange(mesh, it)) {
+				ignore.insert(it2);
+			}
 		}
-		if (borders[current].size() > 0) {
-			borders.emplace_back();
-			current++;
-		}
-	}
-	if (borders.back().size() == 0) {
-		borders.pop_back();
 	}
 	return borders;
 }
 
 
-void setBorderColor(MyMesh &mesh, const HalfedgeStrip &border,
+void setBorderColor(MyMesh &mesh, const HalfedgeHandle &border,
 					MyMesh::Color color) {
-	for (MyMesh::HalfedgeHandle it : border) {
+	for (HalfedgeHandle it : HalfedgeLoopRange(mesh, border)) {
 		mesh.set_color(mesh.to_vertex_handle(it), color);
 	}
 }
+
+
+void fillHoleDumb(MyMesh &mesh, HalfedgeHandle &border) {
+	mesh.request_vertex_status();
+	mesh.request_edge_status();
+	mesh.request_face_status();
+
+	Point center(0, 0, 0);
+	size_t length = 0;
+	for (HalfedgeHandle it : HalfedgeLoopRange(mesh, border)) {
+		VertexHandle vert = mesh.to_vertex_handle(it);
+		center += mesh.point(vert);
+		length++;
+	}
+	center /= length;
+	VertexHandle center_handle = mesh.new_vertex_dirty(center);
+
+	for (HalfedgeHandle it = border;
+		 mesh.to_vertex_handle(it) != center_handle;) {
+		HalfedgeHandle next = mesh.next_halfedge_handle(it);
+		mesh.add_face(mesh.from_vertex_handle(it),
+					  mesh.to_vertex_handle(it),
+					  center_handle);
+		it = next;
+	}
+}

src/mesh_processing.h

@@ -6,11 +6,12 @@
 #include <vector>
 
 
-typedef std::vector<MyMesh::HalfedgeHandle> HalfedgeStrip;
+std::vector<HalfedgeHandle> findBorders(MyMesh &mesh);
 
-std::vector<HalfedgeStrip> findBorders(MyMesh &mesh);
+void setBorderColor(MyMesh &mesh, const HalfedgeHandle &border,
-
-void setBorderColor(MyMesh &mesh, const HalfedgeStrip &border,
 					MyMesh::Color color);
 
+void fillHoleDumb(MyMesh &mesh, HalfedgeHandle &border);
+
+
 #endif

src/my_mesh.h

@@ -14,6 +14,27 @@ struct MyTraits : public OpenMesh::DefaultTraits {
 	typedef OpenMesh::Vec3f Color;
 };
 typedef OpenMesh::PolyMesh_ArrayKernelT<MyTraits> MyMesh;
+typedef MyMesh::FaceHandle FaceHandle;
+typedef MyMesh::VertexHandle VertexHandle;
+typedef MyMesh::HalfedgeHandle HalfedgeHandle;
+typedef MyMesh::EdgeHandle EdgeHandle;
+typedef MyMesh::Point Point;
+
+
+class HalfedgeLoopRange {
+	MyMesh &mesh;
+	const HalfedgeHandle &start;
+
+  public:
+	HalfedgeLoopRange(MyMesh &mesh, const HalfedgeHandle &start)
+		:mesh(mesh), start(start) {}
+	MyMesh::HalfedgeLoopIter begin() {
+		return mesh.hl_begin(start);
+	}
+	MyMesh::HalfedgeLoopIter end() {
+		return mesh.hl_end(start);
+	}
+};
 
 
 #endif