diff options
author | Frederico Linhares <fred@linhares.blue> | 2023-09-28 16:02:41 -0300 |
---|---|---|
committer | Frederico Linhares <fred@linhares.blue> | 2023-09-28 16:02:41 -0300 |
commit | fd4279d77fdc64a534a2bbc05b2f5d3852c765f7 (patch) | |
tree | 196797f3aa21168ad417e9d95ca6779c47f7f155 /src/vk/graphics_pipeline_3d_layout.cpp | |
parent | 6899fac50a2992053e6eea627cb83da1d40e3d79 (diff) |
refa Remove redundant descriptor set layout
* src/vk/graphics_pipeline_3d_layout.cpp: skeletal_model and
static_model were identical, therefore, redundant.
Diffstat (limited to 'src/vk/graphics_pipeline_3d_layout.cpp')
-rw-r--r-- | src/vk/graphics_pipeline_3d_layout.cpp | 55 |
1 files changed, 6 insertions, 49 deletions
diff --git a/src/vk/graphics_pipeline_3d_layout.cpp b/src/vk/graphics_pipeline_3d_layout.cpp index cbb73e5..644aac9 100644 --- a/src/vk/graphics_pipeline_3d_layout.cpp +++ b/src/vk/graphics_pipeline_3d_layout.cpp @@ -103,7 +103,7 @@ unload_descriptor_set_view(void *obj) } void -load_descriptor_set_skeletal_model(void *obj) +load_descriptor_set_model(void *obj) { auto self = static_cast<VK::GraphicsPipeline3DLayout*>(obj); @@ -129,61 +129,19 @@ load_descriptor_set_skeletal_model(void *obj) if(vkCreateDescriptorSetLayout( cg_core.vk_device_with_swapchain->device, &layout_info, nullptr, - &self->descriptor_set_skeletal_model) != VK_SUCCESS) + &self->descriptor_set_model) != VK_SUCCESS) throw CommandError{ "Failed to create Vulkan descriptor set layout for model instance."}; } void -unload_descriptor_set_skeletal_model(void *obj) +unload_descriptor_set_model(void *obj) { auto self = static_cast<VK::GraphicsPipeline3DLayout*>(obj); vkDestroyDescriptorSetLayout( cg_core.vk_device_with_swapchain->device, - self->descriptor_set_skeletal_model, nullptr); -} - -void -load_descriptor_set_static_model(void *obj) -{ - auto self = static_cast<VK::GraphicsPipeline3DLayout*>(obj); - - std::array<VkDescriptorSetLayoutBinding, 2> layout_bindings; - layout_bindings[0].binding = 0; - layout_bindings[0].descriptorType = VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER; - layout_bindings[0].descriptorCount = 1; - layout_bindings[0].stageFlags = VK_SHADER_STAGE_VERTEX_BIT; - layout_bindings[0].pImmutableSamplers = nullptr; - layout_bindings[1].binding = 1; - layout_bindings[1].descriptorType = - VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER; - layout_bindings[1].descriptorCount = 1; - layout_bindings[1].stageFlags = VK_SHADER_STAGE_FRAGMENT_BIT; - layout_bindings[1].pImmutableSamplers = nullptr; - - VkDescriptorSetLayoutCreateInfo layout_info{}; - layout_info.sType = VK_STRUCTURE_TYPE_DESCRIPTOR_SET_LAYOUT_CREATE_INFO; - layout_info.pNext = nullptr; - layout_info.flags = 0; - layout_info.bindingCount = static_cast<uint32_t>(layout_bindings.size()); - layout_info.pBindings = layout_bindings.data(); - - if(vkCreateDescriptorSetLayout( - cg_core.vk_device_with_swapchain->device, &layout_info, nullptr, - &self->descriptor_set_static_model) != VK_SUCCESS) - throw CommandError{ - "Failed to create Vulkan descriptor set layout for model instance."}; -} - -void -unload_descriptor_set_static_model(void *obj) -{ - auto self = static_cast<VK::GraphicsPipeline3DLayout*>(obj); - - vkDestroyDescriptorSetLayout( - cg_core.vk_device_with_swapchain->device, - self->descriptor_set_static_model, nullptr); + self->descriptor_set_model, nullptr); } void @@ -194,7 +152,7 @@ load_pipeline(void *obj) std::array<VkDescriptorSetLayout, 3> set_layouts{ self->descriptor_set_world, self->descriptor_set_view, - self->descriptor_set_skeletal_model}; + self->descriptor_set_model}; VkPipelineLayoutCreateInfo pipeline_layout_info{}; pipeline_layout_info.sType = VK_STRUCTURE_TYPE_PIPELINE_LAYOUT_CREATE_INFO; @@ -307,8 +265,7 @@ unload_render_pass(void *obj) const CommandChain loader{ {&load_descriptor_set_world, &unload_descriptor_set_world}, {&load_descriptor_set_view, &unload_descriptor_set_view}, - {&load_descriptor_set_skeletal_model, &unload_descriptor_set_skeletal_model}, - {&load_descriptor_set_static_model, &unload_descriptor_set_static_model}, + {&load_descriptor_set_model, &unload_descriptor_set_model}, {&load_pipeline, &unload_pipeline}, {&load_render_pass, &unload_render_pass} }; |