X-Git-Url: http://git.localhorst.tv/?a=blobdiff_plain;f=src%2Fmodel.cpp;h=d26968a0e6282e51cdcb9dda431eb90a751d4c33;hb=88620c3c816c47b32a90758f40efe5d02c50bbfa;hp=08b29242b1b185a90c60d54ec59e3b58b2fa4edf;hpb=eca1fdcc8e34a4918418b2de122c6200aeb7ceaf;p=blank.git

diff --git a/src/model.cpp b/src/model.cpp
index 08b2924..d26968a 100644
--- a/src/model.cpp
+++ b/src/model.cpp
@@ -6,23 +6,22 @@
 namespace blank {
 
 Model::Model()
-: vertices()
-, colors()
-, normals()
+:  va(0)
 , handle{}
-, dirty(false) {
+, count(0) {
+	glGenVertexArrays(1, &va);
 	glGenBuffers(ATTRIB_COUNT, handle);
 }
 
 Model::~Model() {
 	glDeleteBuffers(ATTRIB_COUNT, handle);
+	glDeleteVertexArrays(1, &va);
 }
 
 Model::Model(Model &&other)
-: vertices(std::move(other.vertices))
-, colors(std::move(other.colors))
-, normals(std::move(other.normals))
-, dirty(other.dirty) {
+: va(other.va)
+, count(other.count) {
+	other.va = 0;
 	for (int i = 0; i < ATTRIB_COUNT; ++i) {
 		handle[i] = other.handle[i];
 		other.handle[i] = 0;
@@ -30,64 +29,19 @@ Model::Model(Model &&other)
 }
 
 Model &Model::operator =(Model &&other) {
-	vertices = std::move(other.vertices);
-	colors = std::move(other.colors);
-	normals = std::move(other.normals);
+	std::swap(va, other.va);
 	for (int i = 0; i < ATTRIB_COUNT; ++i) {
 		std::swap(handle[i], other.handle[i]);
 	}
-	dirty = other.dirty;
+	count = other.count;
 	return *this;
 }
 
-
-void Model::Clear() {
-	vertices.clear();
-	colors.clear();
-	normals.clear();
-	Invalidate();
-}
-
-void Model::Reserve(int s) {
-	vertices.reserve(s);
-	colors.reserve(s);
-	normals.reserve(s);
-}
-
-
-void Model::Update() {
+void Model::Update(const Buffer &buf) {
+	glBindVertexArray(va);
 	glBindBuffer(GL_ARRAY_BUFFER, handle[ATTRIB_VERTEX]);
-	glBufferData(GL_ARRAY_BUFFER, vertices.size() * sizeof(glm::vec3), vertices.data(), GL_STATIC_DRAW);
-
-#ifndef NDEBUG
-	if (colors.size() < vertices.size()) {
-		std::cerr << "Model: not enough colors!" << std::endl;
-		colors.resize(vertices.size(), { 1, 0, 1 });
-	}
-#endif
-	glBindBuffer(GL_ARRAY_BUFFER, handle[ATTRIB_COLOR]);
-	glBufferData(GL_ARRAY_BUFFER, colors.size() * sizeof(glm::vec3), colors.data(), GL_STATIC_DRAW);
-
-#ifndef NDEBUG
-	if (normals.size() < vertices.size()) {
-		std::cerr << "Model: not enough normals!" << std::endl;
-		normals.resize(vertices.size(), { 0, 1, 0 });
-	}
-#endif
-	glBindBuffer(GL_ARRAY_BUFFER, handle[ATTRIB_NORMAL]);
-	glBufferData(GL_ARRAY_BUFFER, normals.size() * sizeof(glm::vec3), normals.data(), GL_STATIC_DRAW);
-
-	dirty = false;
-}
-
-
-void Model::Draw() {
-	if (dirty) {
-		Update();
-	}
-
+	glBufferData(GL_ARRAY_BUFFER, buf.vertices.size() * sizeof(glm::vec3), buf.vertices.data(), GL_STATIC_DRAW);
 	glEnableVertexAttribArray(ATTRIB_VERTEX);
-	glBindBuffer(GL_ARRAY_BUFFER, handle[ATTRIB_VERTEX]);
 	glVertexAttribPointer(
 		ATTRIB_VERTEX, // location (for shader)
 		3,             // size
@@ -97,8 +51,14 @@ void Model::Draw() {
 		nullptr        // offset
 	);
 
-	glEnableVertexAttribArray(ATTRIB_COLOR);
+#ifndef NDEBUG
+	if (buf.colors.size() < buf.vertices.size()) {
+		std::cerr << "Model: not enough colors!" << std::endl;
+	}
+#endif
 	glBindBuffer(GL_ARRAY_BUFFER, handle[ATTRIB_COLOR]);
+	glBufferData(GL_ARRAY_BUFFER, buf.colors.size() * sizeof(glm::vec3), buf.colors.data(), GL_STATIC_DRAW);
+	glEnableVertexAttribArray(ATTRIB_COLOR);
 	glVertexAttribPointer(
 		ATTRIB_COLOR,  // location (for shader)
 		3,             // size
@@ -108,8 +68,14 @@ void Model::Draw() {
 		nullptr        // offset
 	);
 
-	glEnableVertexAttribArray(ATTRIB_NORMAL);
+#ifndef NDEBUG
+	if (buf.normals.size() < buf.vertices.size()) {
+		std::cerr << "Model: not enough normals!" << std::endl;
+	}
+#endif
 	glBindBuffer(GL_ARRAY_BUFFER, handle[ATTRIB_NORMAL]);
+	glBufferData(GL_ARRAY_BUFFER, buf.normals.size() * sizeof(glm::vec3), buf.normals.data(), GL_STATIC_DRAW);
+	glEnableVertexAttribArray(ATTRIB_NORMAL);
 	glVertexAttribPointer(
 		ATTRIB_NORMAL, // location (for shader)
 		3,             // size
@@ -119,67 +85,59 @@ void Model::Draw() {
 		nullptr        // offset
 	);
 
-	glDrawArrays(
-		GL_TRIANGLES,   // how
-		0,              // start
-		vertices.size() // len
-	);
+	glBindBuffer(GL_ELEMENT_ARRAY_BUFFER, handle[ATTRIB_INDEX]);
+	glBufferData(GL_ELEMENT_ARRAY_BUFFER, buf.indices.size() * sizeof(Index), buf.indices.data(), GL_STATIC_DRAW);
+	count = buf.indices.size();
+}
+
 
-	glDisableVertexAttribArray(ATTRIB_NORMAL);
-	glDisableVertexAttribArray(ATTRIB_COLOR);
-	glDisableVertexAttribArray(ATTRIB_VERTEX);
+void Model::Draw() const {
+	glBindVertexArray(va);
+	glBindBuffer(GL_ELEMENT_ARRAY_BUFFER, handle[ATTRIB_INDEX]);
+	glDrawElements(
+		GL_TRIANGLES,    // how
+		count,           // count
+		GL_UNSIGNED_INT, // type
+		nullptr          // offset
+	);
 }
 
 
 OutlineModel::OutlineModel()
 : vertices()
 , colors()
+, indices()
+, va(0)
 , handle{}
 , dirty(false) {
+	glGenVertexArrays(1, &va);
 	glGenBuffers(ATTRIB_COUNT, handle);
 }
 
 OutlineModel::~OutlineModel() {
 	glDeleteBuffers(ATTRIB_COUNT, handle);
+	glDeleteVertexArrays(1, &va);
 }
 
 
 void OutlineModel::Clear() {
 	vertices.clear();
 	colors.clear();
+	indices.clear();
 	Invalidate();
 }
 
-void OutlineModel::Reserve(int s) {
-	vertices.reserve(s);
-	colors.reserve(s);
+void OutlineModel::Reserve(int v, int i) {
+	vertices.reserve(v);
+	colors.reserve(v);
+	indices.reserve(i);
 }
 
 
 void OutlineModel::Update() {
 	glBindBuffer(GL_ARRAY_BUFFER, handle[ATTRIB_VERTEX]);
 	glBufferData(GL_ARRAY_BUFFER, vertices.size() * sizeof(glm::vec3), vertices.data(), GL_STATIC_DRAW);
-
-#ifndef NDEBUG
-	if (colors.size() < vertices.size()) {
-		std::cerr << "OutlineModel: not enough colors!" << std::endl;
-		colors.resize(vertices.size(), { 1, 0, 1 });
-	}
-#endif
-	glBindBuffer(GL_ARRAY_BUFFER, handle[ATTRIB_COLOR]);
-	glBufferData(GL_ARRAY_BUFFER, colors.size() * sizeof(glm::vec3), colors.data(), GL_STATIC_DRAW);
-
-	dirty = false;
-}
-
-
-void OutlineModel::Draw() {
-	if (dirty) {
-		Update();
-	}
-
 	glEnableVertexAttribArray(ATTRIB_VERTEX);
-	glBindBuffer(GL_ARRAY_BUFFER, handle[ATTRIB_VERTEX]);
 	glVertexAttribPointer(
 		ATTRIB_VERTEX, // location (for shader)
 		3,             // size
@@ -189,8 +147,15 @@ void OutlineModel::Draw() {
 		nullptr        // offset
 	);
 
-	glEnableVertexAttribArray(ATTRIB_COLOR);
+#ifndef NDEBUG
+	if (colors.size() < vertices.size()) {
+		std::cerr << "OutlineModel: not enough colors!" << std::endl;
+		colors.resize(vertices.size(), { 1, 0, 1 });
+	}
+#endif
 	glBindBuffer(GL_ARRAY_BUFFER, handle[ATTRIB_COLOR]);
+	glBufferData(GL_ARRAY_BUFFER, colors.size() * sizeof(glm::vec3), colors.data(), GL_STATIC_DRAW);
+	glEnableVertexAttribArray(ATTRIB_COLOR);
 	glVertexAttribPointer(
 		ATTRIB_COLOR,  // location (for shader)
 		3,             // size
@@ -200,17 +165,30 @@ void OutlineModel::Draw() {
 		nullptr        // offset
 	);
 
+	glBindBuffer(GL_ELEMENT_ARRAY_BUFFER, handle[ATTRIB_INDEX]);
+	glBufferData(GL_ELEMENT_ARRAY_BUFFER, indices.size() * sizeof(Index), indices.data(), GL_STATIC_DRAW);
+
+	dirty = false;
+}
+
+
+void OutlineModel::Draw() {
+	glBindVertexArray(va);
+
+	if (dirty) {
+		Update();
+	}
+
 	glEnable(GL_LINE_SMOOTH);
 	glLineWidth(2.0f);
 
-	glDrawArrays(
-		GL_LINES,       // how
-		0,              // start
-		vertices.size() // len
+	glBindBuffer(GL_ELEMENT_ARRAY_BUFFER, handle[ATTRIB_INDEX]);
+	glDrawElements(
+		GL_LINES,          // how
+		indices.size(),    // count
+		GL_UNSIGNED_SHORT, // type
+		nullptr            // offset
 	);
-
-	glDisableVertexAttribArray(ATTRIB_COLOR);
-	glDisableVertexAttribArray(ATTRIB_VERTEX);
 }
 
 }