Changeset d25381b in opengl-game

Timestamp:

Feb 18, 2020, 9:33:34 PM (5 years ago)

Author:

Dmitry Portnoy <dmitry.portnoy@…>

Branches:

feature/imgui-sdl, master, points-test

Children:

860a0da

Parents:

2d87297

git-author:

Dmitry Portnoy <dmitry.portnoy@…> (02/16/20 21:16:53)

git-committer:

Dmitry Portnoy <dmitry.portnoy@…> (02/18/20 21:33:34)

Message:

In VulkanGame, rename fields related to the ubo and ssbo for the model pipeline to end in _modelPipeline rather than scenePipeline

Files:

• vulkan-game.cpp (modified) (4 diffs)
• vulkan-game.hpp (modified) (3 diffs)

vulkan-game.cpp

@@ -200 +200 @@
 
    createBufferSet(sizeof(UBO_VP_mats), VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT,
-      uniformBuffers_scenePipeline, uniformBuffersMemory_scenePipeline, uniformBufferInfoList_scenePipeline);
+      uniformBuffers_modelPipeline, uniformBuffersMemory_modelPipeline, uniformBufferInfoList_modelPipeline);
    // TODO: Calculate the size of this buffer (and all the other SSBOs) based on the number of objects
    createBufferSet(10 * sizeof(SSBO_ModelObject), VK_BUFFER_USAGE_STORAGE_BUFFER_BIT,
-      storageBuffers_scenePipeline, storageBuffersMemory_scenePipeline, storageBufferInfoList_scenePipeline);
+      storageBuffers_modelPipeline, storageBuffersMemory_modelPipeline, storageBufferInfoList_modelPipeline);
 
    modelPipeline.addDescriptorInfo(VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER,
-      VK_SHADER_STAGE_VERTEX_BIT, &uniformBufferInfoList_scenePipeline);
+      VK_SHADER_STAGE_VERTEX_BIT, &uniformBufferInfoList_modelPipeline);
    modelPipeline.addDescriptorInfo(VK_DESCRIPTOR_TYPE_STORAGE_BUFFER,
-      VK_SHADER_STAGE_VERTEX_BIT, &storageBufferInfoList_scenePipeline);
+      VK_SHADER_STAGE_VERTEX_BIT, &storageBufferInfoList_modelPipeline);
 
    modelPipeline.addDescriptorInfo(VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER,
@@ -787 +787 @@
    so_Asteroid.model = asteroidObjects[0].model_transform * asteroidObjects[0].model_base;
 
-   VulkanUtils::copyDataToMemory(device, uniformBuffersMemory_scenePipeline[currentImage], 0, object_VP_mats);
+   VulkanUtils::copyDataToMemory(device, uniformBuffersMemory_modelPipeline[currentImage], 0, object_VP_mats);
 
    for (size_t i = 0; i < modelObjects.size(); i++) {
-      VulkanUtils::copyDataToMemory(device, storageBuffersMemory_scenePipeline[currentImage],
+      VulkanUtils::copyDataToMemory(device, storageBuffersMemory_modelPipeline[currentImage],
          i, so_Object);
    }
@@ -1460 +1460 @@
 
    createBufferSet(sizeof(UBO_VP_mats), VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT,
-      uniformBuffers_scenePipeline, uniformBuffersMemory_scenePipeline, uniformBufferInfoList_scenePipeline);
+      uniformBuffers_modelPipeline, uniformBuffersMemory_modelPipeline, uniformBufferInfoList_modelPipeline);
    createBufferSet(10 * sizeof(SSBO_ModelObject), VK_BUFFER_USAGE_STORAGE_BUFFER_BIT,
-      storageBuffers_scenePipeline, storageBuffersMemory_scenePipeline, storageBufferInfoList_scenePipeline);
+      storageBuffers_modelPipeline, storageBuffersMemory_modelPipeline, storageBufferInfoList_modelPipeline);
 
    modelPipeline.updateRenderPass(renderPass);
@@ -1519 +1519 @@
    vkDestroySwapchainKHR(device, swapChain, nullptr);
 
-   for (size_t i = 0; i < uniformBuffers_scenePipeline.size(); i++) {
-      vkDestroyBuffer(device, uniformBuffers_scenePipeline[i], nullptr);
-      vkFreeMemory(device, uniformBuffersMemory_scenePipeline[i], nullptr);
-   }
-
-   for (size_t i = 0; i < storageBuffers_scenePipeline.size(); i++) {
-      vkDestroyBuffer(device, storageBuffers_scenePipeline[i], nullptr);
-      vkFreeMemory(device, storageBuffersMemory_scenePipeline[i], nullptr);
+   for (size_t i = 0; i < uniformBuffers_modelPipeline.size(); i++) {
+      vkDestroyBuffer(device, uniformBuffers_modelPipeline[i], nullptr);
+      vkFreeMemory(device, uniformBuffersMemory_modelPipeline[i], nullptr);
+   }
+
+   for (size_t i = 0; i < storageBuffers_modelPipeline.size(); i++) {
+      vkDestroyBuffer(device, storageBuffers_modelPipeline[i], nullptr);
+      vkFreeMemory(device, storageBuffersMemory_modelPipeline[i], nullptr);
    }
 

vulkan-game.hpp

@@ -148 +148 @@
 
       GraphicsPipeline_Vulkan<OverlayVertex, void*> overlayPipeline;
-
       vector<SceneObject<OverlayVertex, void*>> overlayObjects;
 
-      // TODO: Rename all the variables related to modelPipeline to use the same pipelie name
-
       GraphicsPipeline_Vulkan<ModelVertex, SSBO_ModelObject> modelPipeline;
-
       vector<SceneObject<ModelVertex, SSBO_ModelObject>> modelObjects;
 
-      vector<VkBuffer> uniformBuffers_scenePipeline;
-      vector<VkDeviceMemory> uniformBuffersMemory_scenePipeline;
-      vector<VkDescriptorBufferInfo> uniformBufferInfoList_scenePipeline;
-
-      vector<VkBuffer> storageBuffers_scenePipeline;
-      vector<VkDeviceMemory> storageBuffersMemory_scenePipeline;
-      vector<VkDescriptorBufferInfo> storageBufferInfoList_scenePipeline;
+      vector<VkBuffer> uniformBuffers_modelPipeline;
+      vector<VkDeviceMemory> uniformBuffersMemory_modelPipeline;
+      vector<VkDescriptorBufferInfo> uniformBufferInfoList_modelPipeline;
+
+      vector<VkBuffer> storageBuffers_modelPipeline;
+      vector<VkDeviceMemory> storageBuffersMemory_modelPipeline;
+      vector<VkDescriptorBufferInfo> storageBufferInfoList_modelPipeline;
 
       UBO_VP_mats object_VP_mats;
@@ -169 +165 @@
 
       GraphicsPipeline_Vulkan<ShipVertex, SSBO_ModelObject> shipPipeline;
-
       vector<SceneObject<ShipVertex, SSBO_ModelObject>> shipObjects;
 
@@ -184 +179 @@
 
       GraphicsPipeline_Vulkan<AsteroidVertex, SSBO_Asteroid> asteroidPipeline;
-
       vector<SceneObject<AsteroidVertex, SSBO_Asteroid>> asteroidObjects;