break everything

src/main.cpp

@@ -1,16 +1,39 @@
 #include "main_window.h"
+#include "mesh_processor.h"
 #include <iostream>
+#include <QSurfaceFormat>
 #include <QApplication>
 
 
 int main(int argc, char *argv[]) {
 	using namespace std;
+	QSurfaceFormat format;
+	format.setRenderableType(QSurfaceFormat::RenderableType::OpenGL);
+	format.setMajorVersion(2);
+	format.setMinorVersion(1);
+	QSurfaceFormat::setDefaultFormat(format);
 	QApplication app(argc, argv);
-	if (argc > 2) {
-		cerr << "Utilisation : " << argv[0] << " [fichier OBJ]" << endl;
-		return 1;
+	MeshProcessor *mesh_processor = nullptr;
+	MainWindow main_window;
+	MeshViewer *mesh_viewer = &main_window.mesh_viewer;
+	QObject::connect(mesh_viewer, &MeshViewer::initialized,
+					 [&]() {
+						 if (mesh_processor) {
+							 mesh_viewer->addMesh(mesh_processor->mesh);
 						 }
-	MainWindow mw;
-	mw.show();
+					 });
+
+	if (argc > 2) {
+		qWarning("Utilisation : %s [MAILLAGE]", argv[0]);
+		return 1;
+	} else if (argc == 2) {
+		mesh_processor = new MeshProcessor(argv[1]);
+	}
+	// QObject::connect(&main_window, &MainWindow::open,
+	// 				 [&](const QString &path) {
+	// 					 if (mesh_processor) delete mesh_processor;
+	// 					 mesh_processor = new MeshProcessor(path);
+	// 				 });
+	main_window.show();
 	return app.exec();
 }

src/main_window.cpp

@@ -1,5 +1,5 @@
 #include "main_window.h"
-#include "mesh_processing.h"
+#include "mesh_processor.h"
 
 #include <QApplication>
 #include <QFileDialog>
@@ -13,7 +13,7 @@ MainWindow::MainWindow(QWidget *parent)
 	setCentralWidget(&mesh_viewer);
 	addToolBar(Qt::RightToolBarArea, &toolbar);
 	open_action = toolbar.addAction("Ouvrir…", [&](){
-		open(QFileDialog::getOpenFileName(this, "Ouvrir un fichier OBJ"));
+		emit open(QFileDialog::getOpenFileName(this, "Ouvrir un maillage"));
 	});
 	// toolbar_actions.append(toolbar.addAction("Fractionner", [&](){
 	// 	QVector<QPair<MyMesh::Point, MyMesh>> fragments = shatter(mesh);
@@ -33,33 +33,6 @@ MainWindow::MainWindow(QWidget *parent)
 }
 
 
-void MainWindow::open(const QString &path) {
-	OpenMesh::IO::Options options;
-	options.set(OpenMesh::IO::Options::VertexColor);
-	if (!OpenMesh::IO::read_mesh(mesh, path.toUtf8().constData(), options)) {
-		qWarning() << "Failed to read" << path;
-		return;
-	}
-	for (const VertexHandle &vh : mesh.vertices()) {
-		mesh.set_color(vh, MyMesh::Color(0, 0, 0));
-	}
-
- 	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);
-	}
-}
-
-
 void MainWindow::meshViewerInitialized() {
-	if (qApp->arguments().size() == 2) {
-		open(qApp->arguments().at(1));
-	}
 	open_action->setEnabled(true);
 }

src/main_window.h

@@ -11,21 +11,20 @@
 class MainWindow : public QMainWindow {
 	Q_OBJECT
 
-public:
-	MainWindow(QWidget *parent=nullptr);
-	void open(const QString &path);
-
-private slots:
-	void meshViewerInitialized();
-
-private:
-	MyMesh mesh;
-	MeshViewer mesh_viewer;
-	OpenGLMesh *glm = nullptr;
 	QToolBar toolbar;
 	QAction *open_action;
 	QList<QAction *> toolbar_actions;
 
+private slots:
+	void meshViewerInitialized();
+
+signals:
+	void open(const QString &path);
+
+public:
+	MeshViewer mesh_viewer;
+
+	MainWindow(QWidget *parent=nullptr);
 };
 
 

src/mesh_processing.cpp

@@ -1,4 +1,5 @@
 #include "mesh_processing.h"
+#include "util.h"
 #include <QGenericMatrix>
 #include <unordered_set>
 

src/mesh_processing.h

@@ -1,17 +0,0 @@
-#ifndef MESH_PROCESSING_H
-#define MESH_PROCESSING_H
-
-#include "my_mesh.h"
-
-#include <vector>
-
-
-std::vector<HalfedgeHandle> findBorders(MyMesh &mesh);
-
-void setBorderColor(MyMesh &mesh, const HalfedgeHandle &border,
-					MyMesh::Color color);
-
-void fillHoleDumb(MyMesh &mesh, HalfedgeHandle &border);
-
-
-#endif

src/mesh_processor.cpp

@@ -0,0 +1,84 @@
+#include "mesh_processor.h"
+#include "util.h"
+#include <QGenericMatrix>
+#include <unordered_set>
+
+
+static std::vector<HalfedgeHandle> findHoles(MyMesh &mesh) {
+	std::vector<HalfedgeHandle> holes;
+	std::set<HalfedgeHandle> ignore;
+	for (HalfedgeHandle it : mesh.halfedges()) {
+		if (mesh.is_boundary(it)
+			&& ignore.find(it) == ignore.end()) {
+			holes.push_back(it);
+			ignore.insert(it);
+			for (HalfedgeHandle it2 : HalfedgeLoopRange(mesh, it)) {
+				ignore.insert(it2);
+			}
+		}
+	}
+	return holes;
+}
+
+
+MeshProcessor::MeshProcessor(const QString &path) {
+	OpenMesh::IO::Options options;
+	options.set(OpenMesh::IO::Options::VertexColor);
+	if (!OpenMesh::IO::read_mesh(mesh, path.toUtf8().constData(), options)) {
+		qWarning() << "Failed to read" << path;
+		return;
+	}
+	for (const VertexHandle &vh : mesh.vertices()) {
+		mesh.set_color(vh, MyMesh::Color(0, 0, 0));
+	}
+	holes = findHoles(mesh);
+	highlightHoles();
+	fillHoles();
+}
+
+
+static void setHoleBoundaryColor(MyMesh &mesh,
+								 const HalfedgeHandle &hole,
+								 MyMesh::Color color) {
+	for (HalfedgeHandle it : HalfedgeLoopRange(mesh, hole)) {
+		mesh.set_color(mesh.to_vertex_handle(it), color);
+	}
+}
+
+void MeshProcessor::highlightHoles() {
+	for (auto hole : holes) {
+		setHoleBoundaryColor(mesh, hole, {1, 0, 0});
+	}
+}
+
+
+void fillHoleDumb(MyMesh &mesh, HalfedgeHandle &hole) {
+	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, hole)) {
+		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 = hole;
+		 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;
+	}
+}
+
+void MeshProcessor::fillHoles() {
+	for (auto hole : holes) {
+		fillHoleDumb(mesh, hole);
+	}
+}

src/mesh_processor.h

@@ -0,0 +1,28 @@
+#ifndef MESH_PROCESSING_H
+#define MESH_PROCESSING_H
+
+#include "my_mesh.h"
+#include "mesh_processor_painter.h"
+#include <vector>
+#include <QObject>
+
+
+class MeshProcessor : public QObject {
+	Q_OBJECT
+
+	std::vector<HalfedgeHandle> holes;
+
+	// friend class MeshProcessorPainter;
+
+public:
+	MyMesh mesh;
+
+	MeshProcessor(const QString &path);
+
+public slots:
+	void highlightHoles();
+	void fillHoles();
+};
+
+
+#endif

src/mesh_processor_painter.cpp

@@ -0,0 +1,6 @@
+#include "mesh_processor_painter.h"
+
+
+void MeshProcessorPainter::paint() {
+	
+}

src/mesh_processor_painter.h

@@ -0,0 +1,15 @@
+#ifndef MESH_PAINTER_H
+#define MESH_PAINTER_H
+
+#include <QObject>
+
+
+class MeshProcessorPainter {
+	Q_OBJECT
+
+public slots:
+	void paint();
+};
+
+
+#endif

src/mesh_view.cpp

@@ -0,0 +1,73 @@
+#include "mesh_view.h"
+#include <QOpenGLFunctions_2_1>
+
+
+MeshView::MeshView(QOpenGLExtraFunctions *glf, const MyMesh &mesh,
+				   int pos_attr, int col_attr)
+	: glf(glf),
+	  mesh(mesh) {
+    GLfloat *verts = new GLfloat[mesh.n_faces() * 3 * 6];
+	size_t i = 0;
+
+	for (const FaceHandle face : mesh.faces()) {
+		for (const VertexHandle vec : mesh.fv_range(face)) {
+			verts[6*i + 0] = mesh.point(vec)[0];
+			verts[6*i + 1] = mesh.point(vec)[1];
+			verts[6*i + 2] = mesh.point(vec)[2];
+			verts[6*i + 3] = mesh.color.red();
+			verts[6*i + 4] = mesh.color.green();
+			verts[6*i + 5] = mesh.color.blue();
+			i++;
+		}
+    }
+
+    delete[] verts;
+
+	GLuint vao, vbo;
+    glf->glGenVertexArrays(1, &vao);
+	glf->glGenBuffers(1, &vbo);
+
+    glf->glBindVertexArray(vao);
+	glf->glBindBuffer(GL_ARRAY_BUFFER, vbo);
+	glf->glBufferData(GL_ARRAY_BUFFER, nverts * 6 * sizeof (GLfloat), verts, GL_DYNAMIC_DRAW);
+	glf->glVertexAttribPointer(pos_attr, 3, GL_FLOAT, GL_FALSE, 6 * sizeof (GLfloat), 0);
+	glf->glEnableVertexAttribArray(pos_attr);
+	glf->glVertexAttribPointer(col_attr, 3, GL_FLOAT, GL_FALSE, 6 * sizeof (GLfloat),
+						  (const void *) (3 * sizeof (GLfloat)));
+	glf->glEnableVertexAttribArray(col_attr);
+
+	glf->glBindVertexArray(0);
+}
+
+
+MeshView::~MeshView() {
+	glf->glDeleteVertexArrays(1, &vao);
+	glf->glDeleteBuffers(1, &vbo);
+}
+
+
+void MeshView::paint(QOpenGLContext *ctx,
+					 int model_attr, int wireframe_attr) const {
+	void (*glPolygonMode)(GLenum, GLenum) = nullptr;
+	glPolygonMode = (typeof glPolygonMode) ctx->getProcAddress("glPolygonMode");
+	if (!glPolygonMode)
+		qWarning("glPolygonMode not available");
+
+	glf->glUniformMatrix4fv(model_attr, 1, GL_FALSE,
+							mesh.transform.constData());
+
+	/* Mesh */
+	glf->glBindVertexArray(vao);
+	glf->glEnable(GL_POLYGON_OFFSET_FILL);
+	glf->glPolygonOffset(1.0, 2);
+	glf->glDrawArrays(GL_TRIANGLES, 0, nverts);
+	if (glPolygonMode) glPolygonMode(GL_FRONT_AND_BACK, GL_LINE);
+	glf->glDisable(GL_POLYGON_OFFSET_FILL);
+
+	/* Wireframe */
+	glf->glUniform1f(wireframe_attr, 1);
+	glf->glDrawArrays(GL_TRIANGLES, 0, nverts);
+	if (glPolygonMode) glPolygonMode(GL_FRONT_AND_BACK, GL_FILL);
+	glf->glLineWidth(3);
+	glf->glUniform1f(wireframe_attr, 0);
+}

src/mesh_view.h

@@ -0,0 +1,26 @@
+#ifndef MESH_VIEW_H
+#define MESH_VIEW_H
+
+#include "my_mesh.h"
+#include <QOpenGLExtraFunctions>
+#include <QOpenGLContext>
+
+
+class MeshView {
+	GLuint vao;
+	GLuint vbo;
+	size_t nverts;
+	QOpenGLExtraFunctions *glf;
+
+public:
+	const MyMesh &mesh;
+
+	MeshView(QOpenGLExtraFunctions *glf, const MyMesh &mesh,
+			 int pos_attr, int col_attr);
+	~MeshView();
+	void paint(QOpenGLContext *ctx,
+			   int model_attr, int wireframe_attr) const;
+};
+
+
+#endif

src/mesh_viewer.cpp

@@ -1,7 +1,7 @@
 #include "mesh_viewer.h"
 
 #include <tgmath.h>
-#include <utility>
+#include <algorithm>
 
 
 const GLchar *vertex_shader_source = R"glsl(
@@ -69,10 +69,9 @@ opengl_debug_cb(GLenum source,
 void MeshViewer::initializeGL() {
 	// initializeOpenGLFunctions();
 
-	GLint major, minor;
-	glGetIntegerv(GL_MAJOR_VERSION, &major);
-	glGetIntegerv(GL_MINOR_VERSION, &minor);
-	qDebug("OpenGL version %d.%d", major, minor);
+	qDebug("OpenGL version %d.%d",
+		   	context()->format().majorVersion(),
+			context()->format().minorVersion());
 
 	glEnable(GL_DEBUG_OUTPUT);
 	glDebugMessageCallback(opengl_debug_cb, 0);
@@ -138,6 +137,7 @@ void MeshViewer::initializeGL() {
 	glEnable(GL_DEPTH_TEST);
 	glEnable(GL_MULTISAMPLE);
 
+	qDebug("Mesh viewer initialized");
 	emit initialized();
 }
 
@@ -155,80 +155,28 @@ void MeshViewer::paintGL() {
 	trans.translate(0, 0, -cam_dist);
 	QMatrix4x4 view = trans * rot;
 	glUniformMatrix4fv(view_attr, 1, GL_FALSE, view.data());
-	for (const OpenGLMesh m : meshes) {
-		glUniformMatrix4fv(model_attr, 1, GL_FALSE, m.mat.data());
-
-		/* Mesh */
-		glBindVertexArray(m.vao);
-		glEnable(GL_POLYGON_OFFSET_FILL);
-		glPolygonOffset(1.0, 2);
-		glDrawArrays(GL_TRIANGLES, 0, m.nverts);
-		glPolygonMode(GL_FRONT_AND_BACK, GL_LINE);
-		glDisable(GL_POLYGON_OFFSET_FILL);
-
-		/* Wireframe */
-		glUniform1f(wireframe_attr, 1);
-		glDrawArrays(GL_TRIANGLES, 0, m.nverts);
-		glPolygonMode(GL_FRONT_AND_BACK, GL_FILL);
-		glLineWidth(3);
-		glUniform1f(wireframe_attr, 0);
+	for (const MeshView m : meshes) {
+		m.paint(context(), model_attr, wireframe_attr);
 	}
 }
 
 
-OpenGLMesh *MeshViewer::addOpenGLMesh(size_t nverts, const GLfloat *verts, QMatrix4x4 mat, QColor col) {
+void MeshViewer::addMesh(const MyMesh &mesh) {
 	makeCurrent();
-
-	GLuint vao, vbo;
-    glGenVertexArrays(1, &vao);
-	glGenBuffers(1, &vbo);
-
-    glBindVertexArray(vao);
-	glBindBuffer(GL_ARRAY_BUFFER, vbo);
-	glBufferData(GL_ARRAY_BUFFER, nverts * 6 * sizeof (GLfloat), verts, GL_DYNAMIC_DRAW);
-	glVertexAttribPointer(pos_attr, 3, GL_FLOAT, GL_FALSE, 6 * sizeof (GLfloat), 0);
-	glEnableVertexAttribArray(pos_attr);
-	glVertexAttribPointer(col_attr, 3, GL_FLOAT, GL_FALSE, 6 * sizeof (GLfloat),
-						  (const void *) (3 * sizeof (GLfloat)));
-	glEnableVertexAttribArray(col_attr);
-
-	glBindVertexArray(0);
-
-	meshes.append({vao, vbo, nverts, mat, col});
+	meshes.emplace_back(context()->extraFunctions(), mesh,
+						pos_attr, col_attr);
 	doneCurrent();
 	update();
-	return &meshes.last();
 }
 
 
-OpenGLMesh *MeshViewer::addOpenGLMeshFromOpenMesh(MyMesh* mesh, QMatrix4x4 mat, QColor col) {
-    GLfloat *verts = new GLfloat[mesh->n_faces() * 3 * 6];
-	size_t i = 0;
-
-	for (MyMesh::FaceHandle face : mesh->faces()) {
-		for (MyMesh::VertexHandle vec : mesh->fv_range(face)) {
-			verts[6*i + 0] = mesh->point(vec)[0];
-			verts[6*i + 1] = mesh->point(vec)[1];
-			verts[6*i + 2] = mesh->point(vec)[2];
-			verts[6*i + 3] = mesh->color(vec)[0];
-			verts[6*i + 4] = mesh->color(vec)[1];
-			verts[6*i + 5] = mesh->color(vec)[2];
-			i++;
-		}
-    }
-
-    OpenGLMesh *ret = addOpenGLMesh(i, verts, mat, col);
-    delete[] verts;
-	return ret;
-}
-
-
-void MeshViewer::removeOpenGLMesh(OpenGLMesh *mesh) {
-	glDeleteVertexArrays(1, &mesh->vao);
-	glDeleteBuffers(1, &mesh->vbo);
-	for (int i = 0; i < meshes.size(); i++) {
-		if (&meshes[i] == mesh) meshes.removeAt(i);
-	}
+void MeshViewer::removeMesh(const MyMesh &mesh) {
+	makeCurrent();
+	meshes.remove_if([&](const MeshView &mv) {
+		return &mv.mesh == &mesh;
+	});
+	doneCurrent();
+	update();
 }
 
 

src/mesh_viewer.h

@@ -3,6 +3,8 @@
 
 #define GL_GLEXT_PROTOTYPES
 #include "my_mesh.h"
+#include "mesh_view.h"
+#include <list>
 #include <QMatrix4x4>
 #include <QColor>
 #include <QDebug>
@@ -19,37 +21,27 @@
 using namespace OpenMesh;
 
 
-struct OpenGLMesh {
-	GLuint vao;
-	GLuint vbo;
-	size_t nverts;
-	QMatrix4x4 mat;
-	QColor col;
-};
-
-
 class MeshViewer : public QOpenGLWidget {
     Q_OBJECT
 
+	std::list<MeshView> meshes;
+	GLint pos_attr, col_attr, proj_attr, view_attr, model_attr;
+	GLint wf_col_attr, wireframe_attr, alpha_attr;
+	QMatrix4x4 proj;
+	QMatrix4x4 rot, rot_start;
+	GLfloat cam_dist = 1;
+	QPoint mouse_pos;
+
 public:
     MeshViewer(QWidget *parent=nullptr);
 	virtual QSize sizeHint() const override;
     void initializeGL() override;
     void resizeGL(int w, int h) override;
     void paintGL() override;
-	OpenGLMesh *addOpenGLMesh(size_t nverts, const GLfloat *verts, QMatrix4x4 mat=QMatrix4x4(), QColor col={153, 153, 153});
-	OpenGLMesh *addOpenGLMeshFromOpenMesh(MyMesh* _mesh, QMatrix4x4 mat=QMatrix4x4(), QColor col={153, 153, 153});
-	void removeOpenGLMesh(OpenGLMesh *mesh);
-	void setOpenGLMeshMatrix(OpenGLMesh *mv, QMatrix4x4 mat);
 
-private:
-	QList<OpenGLMesh> meshes;
-	GLint pos_attr, col_attr, proj_attr, view_attr, model_attr;
-	GLint wf_col_attr, wireframe_attr, alpha_attr;
-	QMatrix4x4 proj;
-	QMatrix4x4 rot, rot_start;
-	GLfloat cam_dist = 1;
-	QPoint mouse_pos;
+public slots:
+	void addMesh(const MyMesh &mesh);
+	void removeMesh(const MyMesh &mesh);
 
 protected:
     virtual void mousePressEvent(QMouseEvent *e);

src/my_mesh.h

@@ -4,6 +4,8 @@
 #include <OpenMesh/Core/IO/MeshIO.hh>
 #include <OpenMesh/Core/Mesh/PolyMesh_ArrayKernelT.hh>
 #include <OpenMesh/Core/Geometry/VectorT.hh>
+#include <QMatrix4x4>
+#include <QColor>
 
 
 struct MyTraits : public OpenMesh::DefaultTraits {
@@ -13,7 +15,13 @@ struct MyTraits : public OpenMesh::DefaultTraits {
     EdgeAttributes(OpenMesh::Attributes::Color);
 	typedef OpenMesh::Vec3f Color;
 };
-typedef OpenMesh::PolyMesh_ArrayKernelT<MyTraits> MyMesh;
+
+class MyMesh : public OpenMesh::PolyMesh_ArrayKernelT<MyTraits> {
+public:
+	QMatrix4x4 transform;
+	QColor color;
+};
+
 typedef MyMesh::FaceHandle FaceHandle;
 typedef MyMesh::VertexHandle VertexHandle;
 typedef MyMesh::HalfedgeHandle HalfedgeHandle;
@@ -21,20 +29,4 @@ 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

src/util.h

@@ -0,0 +1,23 @@
+#ifndef UTIL_H
+#define UTIL_H
+
+#include "my_mesh.h"
+
+
+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

tp.pro

@@ -19,9 +19,12 @@ win32 {
 HEADERS += src/my_mesh.h
 HEADERS += src/main_window.h
 HEADERS += src/mesh_viewer.h
-HEADERS += src/mesh_processing.h
+HEADERS += src/mesh_view.h
+HEADERS += src/mesh_processor.h
+HEADERS += src/util.h
 
 SOURCES += src/main.cpp
 SOURCES += src/main_window.cpp
 SOURCES += src/mesh_viewer.cpp
-SOURCES += src/mesh_processing.cpp
+SOURCES += src/mesh_view.cpp
+SOURCES += src/mesh_processor.cpp