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

Timestamp:

Feb 11, 2019, 4:01:08 AM (6 years ago)

Author:

Dmitry Portnoy <dmp1488@…>

Branches:

feature/imgui-sdl, master, points-test

Children:

dd9771c

Parents:

4c7cd57

Message:

Use a ShaderModelGroup for the lasers amd remove BufferInfo.vbo_offset since it's no longer being used

File:

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

new-game.cpp

@@ -106 +106 @@
 struct BufferInfo {
    unsigned int vbo_base;
-   unsigned int vbo_offset;
    unsigned int vbo_capacity;
    unsigned int ubo_base;
@@ -216 +215 @@
 
 void renderScene(map<GLuint, BufferInfo>& shaderBufferInfo,
-                  map<ObjectType, ShaderModelGroup>& modelGroups, GLuint ubo,
-                  GLuint laser_sp, GLuint laser_vao);
+                  map<ObjectType, ShaderModelGroup>& modelGroups, GLuint ubo);
 
 void renderSceneGui();
@@ -460 +458 @@
    map<ObjectType, ShaderModelGroup> modelGroups;
 
-   GLuint laser_sp = loadShaderProgram("./laser.vert", "./laser.frag");
-
-   shaderBufferInfo[laser_sp] = BufferInfo();
-
    modelGroups[TYPE_SHIP] = createModelGroup(
       loadShaderProgram("./ship.vert", "./ship.frag"));
@@ -471 +465 @@
       loadShaderProgram("./asteroid.vert", "./asteroid.frag"));
    shaderBufferInfo[modelGroups[TYPE_ASTEROID].shaderProgram] = BufferInfo(); // temporary
+
+   modelGroups[TYPE_LASER] = createModelGroup(
+      loadShaderProgram("./laser.vert", "./laser.frag"));
+   shaderBufferInfo[modelGroups[TYPE_LASER].shaderProgram] = BufferInfo(); // temporary
 
    modelGroups[TYPE_EXPLOSION] = createModelGroup(
@@ -544 +542 @@
    glVertexAttribIPointer(3, 1, GL_UNSIGNED_INT, 0, NULL);
 
-   GLuint laser_vao = 0;
-   glGenVertexArrays(1, &laser_vao);
-   glBindVertexArray(laser_vao);
+   glBindVertexArray(modelGroups[TYPE_LASER].vao);
 
    glEnableVertexAttribArray(0);
@@ -647 +643 @@
    GLuint asteroid_sp_models_ub_index = glGetUniformBlockIndex(modelGroups[TYPE_ASTEROID].shaderProgram, "models");
 
-   GLuint laser_view_mat_loc = glGetUniformLocation(laser_sp, "view");
-   GLuint laser_proj_mat_loc = glGetUniformLocation(laser_sp, "proj");
-   GLuint laser_color_loc = glGetUniformLocation(laser_sp, "laser_color");
-   GLuint laser_sp_models_ub_index = glGetUniformBlockIndex(laser_sp, "models");
+   GLuint laser_view_mat_loc = glGetUniformLocation(modelGroups[TYPE_LASER].shaderProgram, "view");
+   GLuint laser_proj_mat_loc = glGetUniformLocation(modelGroups[TYPE_LASER].shaderProgram, "proj");
+   GLuint laser_color_loc = glGetUniformLocation(modelGroups[TYPE_LASER].shaderProgram, "laser_color");
+   GLuint laser_sp_models_ub_index = glGetUniformBlockIndex(modelGroups[TYPE_LASER].shaderProgram, "models");
 
    GLuint explosion_start_time_loc = glGetUniformLocation(modelGroups[TYPE_EXPLOSION].shaderProgram, "explosion_start_time");
@@ -673 +669 @@
 
 
-   glUseProgram(laser_sp);
+   glUseProgram(modelGroups[TYPE_LASER].shaderProgram);
    glUniformMatrix4fv(laser_view_mat_loc, 1, GL_FALSE, value_ptr(view_mat));
    glUniformMatrix4fv(laser_proj_mat_loc, 1, GL_FALSE, value_ptr(proj_mat));
    glUniform3f(laser_color_loc, 0.2f, 1.0f, 0.2f);
 
-   glUniformBlockBinding(laser_sp, laser_sp_models_ub_index, ub_binding_point);
+   glUniformBlockBinding(modelGroups[TYPE_LASER].shaderProgram, laser_sp_models_ub_index, ub_binding_point);
    glBindBufferRange(GL_UNIFORM_BUFFER, ub_binding_point, ubo, 0, GL_MAX_UNIFORM_BLOCK_SIZE);
 
@@ -812 +808 @@
 
             leftLaser = createLaser(vec3(-0.21f, -1.19f, 1.76f)+offset, vec3(-0.21f, -1.19f, -3.0f)+offset,
-               vec3(0.0f, 1.0f, 0.0f), 0.03f, laser_sp);
+               vec3(0.0f, 1.0f, 0.0f), 0.03f, modelGroups[TYPE_LASER].shaderProgram);
             addObjectToScene(leftLaser, shaderBufferInfo, modelGroups,
                points_vbo,
@@ -828 +824 @@
 
             rightLaser = createLaser(vec3(0.21f, -1.19f, 1.76f) + offset, vec3(0.21f, -1.19f, -3.0f) + offset,
-               vec3(0.0f, 1.0f, 0.0f), 0.03f, laser_sp);
+               vec3(0.0f, 1.0f, 0.0f), 0.03f, modelGroups[TYPE_LASER].shaderProgram);
             addObjectToScene(rightLaser, shaderBufferInfo, modelGroups,
                points_vbo,
@@ -953 +949 @@
          glUniformMatrix4fv(ship_view_mat_loc, 1, GL_FALSE, value_ptr(view_mat));
 
-         glUseProgram(laser_sp);
+         glUseProgram(modelGroups[TYPE_LASER].shaderProgram);
          glUniformMatrix4fv(laser_view_mat_loc, 1, GL_FALSE, value_ptr(view_mat));
 
@@ -972 +968 @@
             break;
          case STATE_GAME:
-            renderScene(shaderBufferInfo, modelGroups, ubo,
-               laser_sp, laser_vao);
+            renderScene(shaderBufferInfo, modelGroups, ubo);
             renderSceneGui();
             break;
@@ -1336 +1331 @@
    BufferInfo* bufferInfo = &shaderBufferInfo[obj->shader_program];
 
-   unsigned int numPoints =
-      obj->type == TYPE_SHIP ||
-      obj->type == TYPE_ASTEROID ||
-      obj->type == TYPE_EXPLOSION ?
-      modelGroups[obj->type].numPoints :
-      bufferInfo->vbo_offset;
-
    // Check if the buffers aren't large enough to fit the new object and, if so, call
    // populateBuffers() to resize and repopupulate them
-   if (bufferInfo->vbo_capacity < (numPoints + obj->num_points) ||
+   if (bufferInfo->vbo_capacity < (modelGroups[obj->type].numPoints + obj->num_points) ||
       bufferInfo->ubo_capacity < (bufferInfo->ubo_offset + 1)) {
 
@@ -2089 +2077 @@
       shaderBufferInfo[shaderCountIt->first].ubo_base = lastShaderUboCount * 2;
 
-      if (shaderCountIt->first != modelGroups[TYPE_SHIP].shaderProgram &&
-         shaderCountIt->first != modelGroups[TYPE_ASTEROID].shaderProgram &&
-         shaderCountIt->first != modelGroups[TYPE_EXPLOSION].shaderProgram) {
-         shaderBufferInfo[shaderCountIt->first].vbo_offset = 0;
-      }
       shaderBufferInfo[shaderCountIt->first].ubo_offset = 0;
 
@@ -2105 +2088 @@
    map<ObjectType, ShaderModelGroup>::iterator modelGroupIt;
    for (modelGroupIt = modelGroups.begin(); modelGroupIt != modelGroups.end(); modelGroupIt++) {
-      if (modelGroupIt->first == TYPE_SHIP ||
-         modelGroupIt->first == TYPE_ASTEROID ||
-         modelGroupIt->first == TYPE_EXPLOSION) {
-         modelGroups[modelGroupIt->first].numPoints = 0;
-      }
+      modelGroups[modelGroupIt->first].numPoints = 0;
    }
 
@@ -2154 +2133 @@
    BufferInfo* bufferInfo = &shaderBufferInfo[obj.shader_program];
 
-   if (obj.type == TYPE_SHIP ||
-      obj.type == TYPE_ASTEROID ||
-      obj.type == TYPE_EXPLOSION) {
-      obj.vertex_vbo_offset = bufferInfo->vbo_base + modelGroups[obj.type].numPoints;
-   } else {
-      obj.vertex_vbo_offset = bufferInfo->vbo_base + bufferInfo->vbo_offset;
-   }
-   
+   obj.vertex_vbo_offset = bufferInfo->vbo_base + modelGroups[obj.type].numPoints;
    obj.ubo_offset = bufferInfo->ubo_base + bufferInfo->ubo_offset;
 
@@ -2201 +2173 @@
    }
 
-   if (obj.type == TYPE_SHIP ||
-      obj.type == TYPE_ASTEROID ||
-      obj.type == TYPE_EXPLOSION) {
-      modelGroups[obj.type].numPoints += obj.num_points;
-   } else {
-      bufferInfo->vbo_offset += obj.num_points;
-   }
+   modelGroups[obj.type].numPoints += obj.num_points;
    bufferInfo->ubo_offset++;
 }
@@ -2372 +2338 @@
 
 void renderScene(map<GLuint, BufferInfo>& shaderBufferInfo,
-                  map<ObjectType, ShaderModelGroup>& modelGroups, GLuint ubo,
-                  GLuint laser_sp, GLuint laser_vao) {
+                  map<ObjectType, ShaderModelGroup>& modelGroups, GLuint ubo) {
 
    glUseProgram(modelGroups[TYPE_SHIP].shaderProgram);
@@ -2387 +2352 @@
    glEnable(GL_BLEND);
 
-   glUseProgram(laser_sp);
-   glBindVertexArray(laser_vao);
-
-   glDrawArrays(GL_TRIANGLES, shaderBufferInfo[laser_sp].vbo_base, shaderBufferInfo[laser_sp].vbo_offset);
+   glUseProgram(modelGroups[TYPE_LASER].shaderProgram);
+   glBindVertexArray(modelGroups[TYPE_LASER].vao);
+
+   glDrawArrays(GL_TRIANGLES, shaderBufferInfo[modelGroups[TYPE_LASER].shaderProgram].vbo_base, modelGroups[TYPE_LASER].numPoints);
 
    glUseProgram(modelGroups[TYPE_EXPLOSION].shaderProgram);