Changeset a926b79 in opengl-game

Timestamp:

Mar 29, 2019, 6:47:06 PM (6 years ago)

Author:

Dmitry Portnoy <dmitry.portnoy@…>

Branches:

feature/imgui-sdl, master, points-test

Children:

c4c205e

Parents:

b05e2b5

git-author:

Dmitry Portnoy <dmitry.portnoy@…> (03/29/19 18:43:20)

git-committer:

Dmitry Portnoy <dmitry.portnoy@…> (03/29/19 18:47:06)

Message:

For TYPE_SHIP objects, switch to using vbos specific to the model group as opposed to the generic vbos.

File:

• new-game.cpp (modified) (7 diffs)

new-game.cpp

@@ -521 +521 @@
    initModelGroupAttribs(modelGroups[TYPE_SHIP]);
 
-   glBindBuffer(GL_ARRAY_BUFFER, points_vbo);
-   glVertexAttribPointer(0, 3, GL_FLOAT, GL_FALSE, 0, NULL);
-   modelGroups[TYPE_SHIP].attribs["vertex_position"].buffer = points_vbo;
-
-   glBindBuffer(GL_ARRAY_BUFFER, colors_vbo);
-   glVertexAttribPointer(1, 3, GL_FLOAT, GL_FALSE, 0, NULL);
-   modelGroups[TYPE_SHIP].attribs["vertex_color"].buffer = colors_vbo;
-
-   glBindBuffer(GL_ARRAY_BUFFER, normals_vbo);
-   glVertexAttribPointer(2, 3, GL_FLOAT, GL_FALSE, 0, NULL);
-   modelGroups[TYPE_SHIP].attribs["vertex_normal"].buffer = normals_vbo;
-
-   glBindBuffer(GL_ARRAY_BUFFER, model_mat_idx_vbo);
-   glVertexAttribIPointer(3, 1, GL_UNSIGNED_INT, 0, NULL);
-   modelGroups[TYPE_SHIP].attribs["ubo_index"].buffer = model_mat_idx_vbo;
-
    modelGroups[TYPE_ASTEROID] = createModelGroup(
       loadShaderProgram("./asteroid.vert", "./asteroid.frag"));
@@ -635 +619 @@
 
    // player ship
-   SceneObject* ship = createShip();
-   objects.push_back(ship);
+   objects.push_back(createShip());
 
    vector<SceneObject>::iterator obj_it;
@@ -1987 +1970 @@
 }
 
+// TODO: Add the code to resize the buffers here
+// addObjectToScene and removeObjectFromScene pretty much already do this.
+// However, when addObjectToScene resizes the buffers, it resizes them for all object types
+// It would be more efficient to only resize them for the object type in question
+
 void removeModelFromGroup(ShaderModelGroup& modelGroup, SceneObject& model) {
    // TODO: Implement
@@ -2277 +2265 @@
       smg->numPoints = 0;
       smg->vboCapacity = shaderCounts[smg->shaderProgram] * 2;
-      /*
-      if (modelGroupIt->first == TYPE_LASER) {
-         smg = &modelGroups[modelGroupIt->first];
-         smg->numPoints = shaderCounts[smg->shaderProgram];
-
-         // Here, I could add glBufferData calls to allocate space for laser buffers
-         if (modelGroupIt->first == TYPE_LASER) {
-            glBindBuffer(GL_ARRAY_BUFFER, smg->attribs["vertex_position"].buffer);
-            glBufferData(GL_ARRAY_BUFFER, smg->numPoints * sizeof(GLfloat) * smg->attribs["vertex_position"].size, NULL, GL_DYNAMIC_DRAW);
-         }
-      }
-      */
+
+      if (modelGroupIt->first == TYPE_SHIP) {
+         int numPoints = shaderCounts[smg->shaderProgram];
+         AttribInfo* attrib;
+
+         attrib = &smg->attribs["vertex_position"];
+         glBindBuffer(GL_ARRAY_BUFFER, attrib->buffer);
+         glBufferData(GL_ARRAY_BUFFER, numPoints * GLsizeof(attrib->type) * attrib->size, NULL, GL_DYNAMIC_DRAW);
+
+         attrib = &smg->attribs["vertex_color"];
+         glBindBuffer(GL_ARRAY_BUFFER, attrib->buffer);
+         glBufferData(GL_ARRAY_BUFFER, numPoints * GLsizeof(attrib->type) * attrib->size, NULL, GL_DYNAMIC_DRAW);
+
+         attrib = &smg->attribs["vertex_normal"];
+         glBindBuffer(GL_ARRAY_BUFFER, attrib->buffer);
+         glBufferData(GL_ARRAY_BUFFER, numPoints * GLsizeof(attrib->type) * attrib->size, NULL, GL_DYNAMIC_DRAW);
+
+         attrib = &smg->attribs["ubo_index"];
+         glBindBuffer(GL_ARRAY_BUFFER, attrib->buffer);
+         glBufferData(GL_ARRAY_BUFFER, numPoints * GLsizeof(attrib->type) * attrib->size, NULL, GL_DYNAMIC_DRAW);
+      }
    }
 
@@ -2322 +2319 @@
    BufferInfo* bufferInfo = &shaderBufferInfo[modelGroups[obj.type].shaderProgram];
 
-   obj.vertex_vbo_offset = bufferInfo->vbo_base + modelGroups[obj.type].numPoints;
+   if (obj.type == TYPE_SHIP) {
+      obj.vertex_vbo_offset = modelGroups[obj.type].numPoints;
+   } else {
+      obj.vertex_vbo_offset = bufferInfo->vbo_base + modelGroups[obj.type].numPoints;
+   }
    obj.ubo_offset = bufferInfo->ubo_base + bufferInfo->ubo_offset;
 
@@ -2347 +2348 @@
                break;
             case ATTRIB_UNIFORM:
-              break;
+               break;
          }
       }
@@ -2536 +2537 @@
    glBindVertexArray(modelGroups[TYPE_SHIP].vao);
 
-   glDrawArrays(GL_TRIANGLES, shaderBufferInfo[modelGroups[TYPE_SHIP].shaderProgram].vbo_base, modelGroups[TYPE_SHIP].numPoints);
+   glDrawArrays(GL_TRIANGLES, 0, modelGroups[TYPE_SHIP].numPoints);
 
    glUseProgram(modelGroups[TYPE_ASTEROID].shaderProgram);