Changes in graphics-pipeline_vulkan.hpp [e8445f0:5049354] in opengl-game

File:

• graphics-pipeline_vulkan.hpp (modified) (6 diffs)

graphics-pipeline_vulkan.hpp

@@ -32 +32 @@
 };
 
-// TODO: Use this struct for uniform buffers as well and rename it to VulkanBuffer (maybe move it to VulkanUtils)
-// Also, probably better to make this a vector of structs where each struct
-// has a VkBuffer, VkDeviceMemory, and VkDescriptorBufferInfo
+// TODO: Use this struct for uniform buffers as well (maybe move it to VulkanUtils)
 struct StorageBufferSet {
    vector<VkBuffer> buffers;
@@ -66 +64 @@
       void addStorageDescriptor(VkShaderStageFlags stageFlags);
 
-      // TODO: I might be able to use a single VkDescriptorBufferInfo here and reuse it when creating the descriptor sets
-      void addDescriptorInfo(VkDescriptorType type, VkShaderStageFlags stageFlags,
-                             vector<VkDescriptorBufferInfo>* bufferData);
+      void addDescriptorInfo(VkDescriptorType type, VkShaderStageFlags stageFlags, vector<VkDescriptorBufferInfo>* bufferData);
       void addDescriptorInfo(VkDescriptorType type, VkShaderStageFlags stageFlags, VkDescriptorImageInfo* imageData);
 
@@ -79 +75 @@
 
       bool addObject(const vector<VertexType>& vertices, vector<uint16_t> indices, SSBOType& ssbo,
-                     VkCommandPool commandPool, VkQueue graphicsQueue);
+         VkCommandPool commandPool, VkQueue graphicsQueue);
 
       void updateObject(size_t objIndex, SSBOType& ssbo);
 
       void updateObjectVertices(size_t objIndex, const vector<VertexType>& vertices, VkCommandPool commandPool,
-                                VkQueue graphicsQueue);
+         VkQueue graphicsQueue);
 
       void cleanup();
@@ -460 +456 @@
          switch (descriptorWrites[j].descriptorType) {
             case VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER:
-            case VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER_DYNAMIC:
             case VK_DESCRIPTOR_TYPE_STORAGE_BUFFER:
                descriptorWrites[j].pBufferInfo = &(*this->descriptorInfoList[j].bufferDataList)[i];
@@ -479 +474 @@
 void GraphicsPipeline_Vulkan<VertexType, SSBOType>::createRenderCommands(VkCommandBuffer& commandBuffer,
       uint32_t currentImage) {
-
    vkCmdBindPipeline(commandBuffer, VK_PIPELINE_BIND_POINT_GRAPHICS, pipeline);
-
    vkCmdBindDescriptorSets(commandBuffer, VK_PIPELINE_BIND_POINT_GRAPHICS, pipelineLayout, 0, 1,
       &descriptorSets[currentImage], 0, nullptr);
@@ -550 +543 @@
 }
 
-// TODO: Allow a swapchain index to be passed in instead of updating all of them
-// Actually, since I'm in the process of replacing SSBOs with dynamic UBOs, I can ignore that for this function
 template<class VertexType, class SSBOType>
 void GraphicsPipeline_Vulkan<VertexType, SSBOType>::updateObject(size_t objIndex, SSBOType& ssbo) {
    if (!is_same_v<SSBOType, void*>) {
       for (size_t i = 0; i < storageBufferSet.memory.size(); i++) {
-         VulkanUtils::copyDataToMemory(this->device, ssbo, storageBufferSet.memory[i], objIndex * sizeof(SSBOType));
+         VulkanUtils::copyDataToMemory(this->device, storageBufferSet.memory[i], objIndex * sizeof(SSBOType), ssbo);
       }
    }