From e85db91c6b6a326e3339020bc434063db6c957f9 Mon Sep 17 00:00:00 2001
From: Dan Goodliffe <dan@randomdan.homeip.net>
Date: Sat, 20 Feb 2021 15:12:25 +0000
Subject: Fix test-obj by splitting mesh processing

Don't invoke GL parts from test code
---
 gfx/models/obj.impl.cpp | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

(limited to 'gfx/models/obj.impl.cpp')

diff --git a/gfx/models/obj.impl.cpp b/gfx/models/obj.impl.cpp
index 0c1303a..5ac15ac 100644
--- a/gfx/models/obj.impl.cpp
+++ b/gfx/models/obj.impl.cpp
@@ -1,6 +1,6 @@
 #include "obj.h"
 #include <algorithm>
-#include <gfx/models/mesh.h>
+#include <gfx/models/mesh.h> // IWYU pragma: keep
 #include <gfx/models/vertex.hpp>
 #include <glm/glm.hpp>
 #include <iterator>
@@ -12,8 +12,19 @@ std::vector<ObjParser::NamedMesh>
 ObjParser::createMeshes() const
 {
 	std::vector<ObjParser::NamedMesh> out;
+	const auto data {createMeshData()};
+	std::transform(data.begin(), data.end(), std::back_inserter(out), [](auto && obj) {
+		return std::make_pair(obj.first, std::make_shared<Mesh>(obj.second.first, obj.second.second));
+	});
+	return out;
+}
+
+std::vector<ObjParser::NamedMeshData>
+ObjParser::createMeshData() const
+{
+	std::vector<ObjParser::NamedMeshData> out;
 	out.reserve(objects.size());
-	for (const auto & obj : objects) {
+	std::transform(objects.begin(), objects.end(), std::back_inserter(out), [this](auto && obj) {
 		std::vector<Vertex> overtices;
 		std::vector<ObjParser::FaceElement> vertexOrder;
 		std::vector<unsigned int> indices;
@@ -36,7 +47,7 @@ ObjParser::createMeshes() const
 				f(idx - 1);
 			}
 		}
-		out.emplace_back(obj.first, std::make_shared<Mesh>(overtices, indices));
-	}
+		return std::make_pair(obj.first, std::make_pair(overtices, indices));
+	});
 	return out;
 }
-- 
cgit v1.2.3