Changeset a9d191a in opengl-game for new-game.cpp

Timestamp:

Apr 12, 2019, 3:22:33 PM (5 years ago)

Author:

Dmitry Portnoy <dmitry.portnoy@…>

Branches:

feature/imgui-sdl, master, points-test

Children:

14e6918

Parents:

c4c205e

Message:

Fix the buffer resizing algorithm for model groups (this fixes the laser rendering issue)

File:

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

new-game.cpp

@@ -261 +261 @@
 void renderSceneGui(map<string, vector<UIValue>> valueLists);
 
+void initGuiValueLists(map<string, vector<UIValue>> valueLists);
 void renderGuiValueList(vector<UIValue>& values);
 
@@ -308 +309 @@
 SceneObject* objExplosion;
 SceneObject* objFirst;
+
+map<string, vector<UIValue>> valueLists;
 
 /*
@@ -589 +592 @@
    initModelGroupAttribs(modelGroups[TYPE_LASER]);
 
-   glBindBuffer(GL_ARRAY_BUFFER, points_vbo);
-   glVertexAttribPointer(0, 3, GL_FLOAT, GL_FALSE, 0, NULL);
-   modelGroups[TYPE_LASER].attribs["vertex_position"].buffer = points_vbo;
-
-   glBindBuffer(GL_ARRAY_BUFFER, texcoords_vbo);
-   glVertexAttribPointer(1, 2, GL_FLOAT, GL_FALSE, 0, NULL);
-   modelGroups[TYPE_LASER].attribs["vt"].buffer = texcoords_vbo;
-
-   glBindBuffer(GL_ARRAY_BUFFER, model_mat_idx_vbo);
-   glVertexAttribIPointer(2, 1, GL_UNSIGNED_INT, 0, NULL);
-   modelGroups[TYPE_LASER].attribs["ubo_index"].buffer = model_mat_idx_vbo;
-
    modelGroups[TYPE_EXPLOSION] = createModelGroup(
       loadShaderProgram("./explosion.vert", "./explosion.frag"));
@@ -606 +597 @@
 
    // The last parameter (offset) is only used for populating the buffers since the distance
-   // between each item is needed there. However, that code isn't run for explosions right now anyway,
-   // so I may as well pass in 0 here.
+   // between each item is needed there. However, the explosion vbos are populated using different
+   // code anyway and don't use that argument, so I may as well pass in 0 here.
    defineModelGroupAttrib(modelGroups[TYPE_EXPLOSION], "v_i", ATTRIB_POINT_VARYING,
       3, GL_FLOAT, 0);
@@ -783 +774 @@
    State curState = STATE_MAIN_MENU;
 
-   map<string, vector<UIValue>> valueLists;
-
-   valueLists["stats value list"] = vector<UIValue>();
+   initGuiValueLists(valueLists);
+
    valueLists["stats value list"].push_back(UIValue(UIVALUE_INT, "Score", &score));
    valueLists["stats value list"].push_back(UIValue(UIVALUE_DOUBLE, "FPS", &fps));
-
-   valueLists["debug value list"] = vector<UIValue>();
-
-   //valueLists["debug value list"].push_back(UIValue(UIVALUE_INT, "Buffer offset", &bufferOffset));
 
    while (!glfwWindowShouldClose(window) && isRunning) {
@@ -2291 +2277 @@
       smg->vboCapacity = shaderCounts[smg->shaderProgram] * 2;
 
+      AttribInfo* attrib;
+
       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);
+         glBufferData(GL_ARRAY_BUFFER, smg->vboCapacity * 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);
+         glBufferData(GL_ARRAY_BUFFER, smg->vboCapacity * 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);
+         glBufferData(GL_ARRAY_BUFFER, smg->vboCapacity * 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);
+         glBufferData(GL_ARRAY_BUFFER, smg->vboCapacity * GLsizeof(attrib->type) * attrib->size, NULL, GL_DYNAMIC_DRAW);
+      } else if (modelGroupIt->first == TYPE_LASER) {
+         attrib = &smg->attribs["vertex_position"];
+         glBindBuffer(GL_ARRAY_BUFFER, attrib->buffer);
+         glBufferData(GL_ARRAY_BUFFER, smg->vboCapacity * GLsizeof(attrib->type) * attrib->size, NULL, GL_DYNAMIC_DRAW);
+
+         attrib = &smg->attribs["vt"];
+         glBindBuffer(GL_ARRAY_BUFFER, attrib->buffer);
+         glBufferData(GL_ARRAY_BUFFER, smg->vboCapacity * GLsizeof(attrib->type) * attrib->size, NULL, GL_DYNAMIC_DRAW);
+
+         attrib = &smg->attribs["ubo_index"];
+         glBindBuffer(GL_ARRAY_BUFFER, attrib->buffer);
+         glBufferData(GL_ARRAY_BUFFER, smg->vboCapacity * GLsizeof(attrib->type) * attrib->size, NULL, GL_DYNAMIC_DRAW);
       }
    }
@@ -2344 +2341 @@
    BufferInfo* bufferInfo = &shaderBufferInfo[modelGroups[obj.type].shaderProgram];
 
-   if (obj.type == TYPE_SHIP) {
+   if (obj.type == TYPE_SHIP || obj.type == TYPE_LASER) {
       obj.vertex_vbo_offset = modelGroups[obj.type].numPoints;
    } else {
@@ -2574 +2571 @@
    glBindVertexArray(modelGroups[TYPE_LASER].vao);
 
-   glDrawArrays(GL_TRIANGLES, shaderBufferInfo[modelGroups[TYPE_LASER].shaderProgram].vbo_base, modelGroups[TYPE_LASER].numPoints);
+   glDrawArrays(GL_TRIANGLES, 0, modelGroups[TYPE_LASER].numPoints);
 
    glUseProgram(modelGroups[TYPE_EXPLOSION].shaderProgram);
@@ -2699 +2696 @@
 }
 
+void initGuiValueLists(map<string, vector<UIValue>> valueLists) {
+   valueLists["stats value list"] = vector<UIValue>();
+   valueLists["debug value list"] = vector<UIValue>();
+}
+
 void renderGuiValueList(vector<UIValue>& values) {
    float maxWidth = 0.0f;