Changeset 1abebc1 in opengl-game for sdl-game.hpp

Timestamp:

May 19, 2021, 4:49:43 PM (4 years ago)

Author:

Dmitry Portnoy <dportnoy@…>

Branches:

feature/imgui-sdl

Children:

c163d81

Parents:

a3cefaa

Message:

Remove the storageBuffers parameter from addObject() since it is no longer used, rename StorageBufferSet, resizeStorageBufferSet(), and updateStorageuffer() to BufferSet, resizeBufferSet(), and updateBufferSet() respectively, and change updateObject() to just take a SceneObject reference.

File:

• sdl-game.hpp (modified) (9 diffs)

sdl-game.hpp

@@ -79 +79 @@
 // has a VkBuffer, VkDeviceMemory, and VkDescriptorBufferInfo
 // TODO: Maybe change the structure here since VkDescriptorBufferInfo already stores a reference to the VkBuffer
-struct StorageBufferSet {
+struct BufferSet {
    vector<VkBuffer> buffers;
    vector<VkDeviceMemory> memory;
@@ -220 +220 @@
       GraphicsPipeline_Vulkan<ModelVertex> modelPipeline;
 
-      StorageBufferSet storageBuffers_modelPipeline;
+      BufferSet storageBuffers_modelPipeline;
       VulkanBuffer<SSBO_ModelObject> objects_modelPipeline;
 
@@ -299 +299 @@
       // TODO: Remove the need for templating, which is only there so a GraphicsPupeline_Vulkan can be passed in
       template<class VertexType, class SSBOType>
-      void resizeStorageBufferSet(StorageBufferSet& set, VulkanBuffer<SSBOType>& buffer,
-                                  GraphicsPipeline_Vulkan<VertexType>& pipeline,
-                                  VkCommandPool commandPool, VkQueue graphicsQueue);
+      void resizeBufferSet(BufferSet& set, VulkanBuffer<SSBOType>& buffer,
+                           GraphicsPipeline_Vulkan<VertexType>& pipeline, VkCommandPool commandPool,
+                           VkQueue graphicsQueue);
 
       template<class SSBOType>
-      void updateStorageBuffer(StorageBufferSet& storageBufferSet, size_t objIndex, SSBOType& ssbo);
+      void updateBufferSet(BufferSet& set, size_t objIndex, SSBOType& ssbo);
 
       // TODO: Since addObject() returns a reference to the new object now,
@@ -312 +312 @@
                                                    GraphicsPipeline_Vulkan<VertexType>& pipeline,
                                                    const vector<VertexType>& vertices, vector<uint16_t> indices,
-                                                   SSBOType ssbo, StorageBufferSet& storageBuffers);
+                                                   SSBOType ssbo);
 
       template<class VertexType>
@@ -324 +324 @@
 
       template<class VertexType, class SSBOType>
-      void updateObject(vector<SceneObject<VertexType, SSBOType>>& objects,
-                        GraphicsPipeline_Vulkan<VertexType>& pipeline, size_t index);
+      void updateObject(SceneObject<VertexType, SSBOType>& obj);
 
       void renderFrame(ImDrawData* draw_data);
@@ -347 +346 @@
 
 template<class VertexType, class SSBOType>
-void VulkanGame::resizeStorageBufferSet(StorageBufferSet& set, VulkanBuffer<SSBOType>& buffer,
-                                        GraphicsPipeline_Vulkan<VertexType>& pipeline,
-                                        VkCommandPool commandPool, VkQueue graphicsQueue) {
+void VulkanGame::resizeBufferSet(BufferSet& set, VulkanBuffer<SSBOType>& buffer,
+                                 GraphicsPipeline_Vulkan<VertexType>& pipeline, VkCommandPool commandPool,
+                                 VkQueue graphicsQueue) {
    size_t numObjects = buffer.numObjects < buffer.capacity ? buffer.numObjects : buffer.capacity;
 
@@ -388 +387 @@
 // TODO: See if it makes sense to pass in the current swapchain index instead of updating all of them
 template<class SSBOType>
-void VulkanGame::updateStorageBuffer(StorageBufferSet& storageBufferSet, size_t objIndex, SSBOType& ssbo) {
-   for (size_t i = 0; i < storageBufferSet.memory.size(); i++) {
-      VulkanUtils::copyDataToMemory(device, ssbo, storageBufferSet.memory[i], objIndex * sizeof(SSBOType));
+void VulkanGame::updateBufferSet(BufferSet& set, size_t objIndex, SSBOType& ssbo) {
+   for (size_t i = 0; i < set.memory.size(); i++) {
+      VulkanUtils::copyDataToMemory(device, ssbo, set.memory[i], objIndex * sizeof(SSBOType));
    }
 }
@@ -401 +400 @@
                                                          GraphicsPipeline_Vulkan<VertexType>& pipeline,
                                                          const vector<VertexType>& vertices, vector<uint16_t> indices,
-                                                         SSBOType ssbo, StorageBufferSet& storageBuffers) {
+                                                         SSBOType ssbo) {
    // TODO: Use the model field of ssbo to set the object's model_base
    // currently, the passed in model is useless since it gets overridden in updateObject() anyway
@@ -504 +503 @@
 // TODO: Just pass in the single object instead of a list of all of them
 template<class VertexType, class SSBOType>
-void VulkanGame::updateObject(vector<SceneObject<VertexType, SSBOType>>& objects,
-                              GraphicsPipeline_Vulkan<VertexType>& pipeline, size_t index) {
-   SceneObject<VertexType, SSBOType>& obj = objects[index];
-
+void VulkanGame::updateObject(SceneObject<VertexType, SSBOType>& obj) {
    obj.ssbo.model = obj.model_transform * obj.model_base;
    obj.center = vec3(obj.ssbo.model * vec4(0.0f, 0.0f, 0.0f, 1.0f));