Changeset f94eea9 in opengl-game

Timestamp:

Sep 27, 2019, 7:20:55 PM (5 years ago)

Author:

Dmitry Portnoy <dmitry.portnoy@…>

Branches:

feature/imgui-sdl, master, points-test

Children:

6fc24c7

Parents:

054d9ed

Message:

In vulkangame, add code to create image views

Files:

• vulkan-game.cpp (modified) (4 diffs)
• vulkan-game.hpp (modified) (2 diffs)
• vulkan-ref.cpp (modified) (10 diffs)
• vulkan-utils.cpp (modified) (1 diff)
• vulkan-utils.hpp (modified) (1 diff)

vulkan-game.cpp

@@ -101 +101 @@
    createLogicalDevice(validationLayers, deviceExtensions);
    createSwapChain();
+   createImageViews();
 }
 
@@ -175 +176 @@
    gui->shutdown();
    delete gui;
-}
-
-void VulkanGame::cleanupSwapChain() {
-   vkDestroySwapchainKHR(device, swapChain, nullptr);
 }
 
@@ -389 +386 @@
       createInfo.queueFamilyIndexCount = 2;
       createInfo.pQueueFamilyIndices = queueFamilyIndices;
-   }
-   else {
+   } else {
       createInfo.imageSharingMode = VK_SHARING_MODE_EXCLUSIVE;
       createInfo.queueFamilyIndexCount = 0;
@@ -413 +409 @@
    swapChainExtent = extent;
 }
+
+void VulkanGame::createImageViews() {
+   swapChainImageViews.resize(swapChainImages.size());
+
+   for (size_t i = 0; i < swapChainImages.size(); i++) {
+      swapChainImageViews[i] = VulkanUtils::createImageView(device, swapChainImages[i], swapChainImageFormat,
+         VK_IMAGE_ASPECT_COLOR_BIT);
+   }
+}
+
+void VulkanGame::cleanupSwapChain() {
+   for (auto imageView : swapChainImageViews) {
+      vkDestroyImageView(device, imageView, nullptr);
+   }
+
+   vkDestroySwapchainKHR(device, swapChain, nullptr);
+}

vulkan-game.hpp

@@ -37 +37 @@
       VkFormat swapChainImageFormat;
       VkExtent2D swapChainExtent;
+      vector<VkImageView> swapChainImageViews;
 
       bool initWindow(int width, int height, unsigned char guiFlags);
@@ -55 +56 @@
          const vector<const char*>& deviceExtensions);
       void createSwapChain();
+      void createImageViews();
+
       void cleanupSwapChain();
 

vulkan-ref.cpp

@@ -173 +173 @@
       VkFormat swapChainImageFormat;
       VkExtent2D swapChainExtent;
+      vector<VkImageView> swapChainImageViews;
 /*** END OF REFACTORED CODE ***/
-      vector<VkImageView> swapChainImageViews;
       vector<VkFramebuffer> swapChainFramebuffers;
 
@@ -317 +317 @@
          pickPhysicalDevice();
          createLogicalDevice();
-/*** END OF REFACTORED CODE ***/
          createSwapChain();
          createImageViews();
+/*** END OF REFACTORED CODE ***/
          createRenderPass();
 
@@ -685 +685 @@
          return details;
       }
-/*** END OF REFACTORED CODE ***/
 
       VkSurfaceFormatKHR chooseSwapSurfaceFormat(const vector<VkSurfaceFormatKHR>& availableFormats) {
@@ -735 +734 @@
          }
       }
+/*** END OF REFACTORED CODE ***/
 
       void createRenderPass() {
@@ -1330 +1330 @@
       }
 
+/*** START OF REFACTORED CODE ***/
       VkImageView createImageView(VkImage image, VkFormat format, VkImageAspectFlags aspectFlags) {
          VkImageViewCreateInfo viewInfo = {};
@@ -1355 +1356 @@
          return imageView;
       }
+/*** END OF REFACTORED CODE ***/
 
       void createTextureSampler() {
@@ -1708 +1710 @@
          vkDeviceWaitIdle(device);
       }
+
+      void drawFrame() {
 /*** END OF REFACTORED CODE ***/
-
-      void drawFrame() {
          vkWaitForFences(device, 1, &inFlightFences[currentFrame], VK_TRUE, numeric_limits<uint64_t>::max());
 
@@ -1772 +1774 @@
          currentFrame = (currentFrame + 1) % MAX_FRAMES_IN_FLIGHT;
          currentFrame = (currentFrame + 1) % MAX_FRAMES_IN_FLIGHT;
+/*** START OF REFACTORED CODE ***/
       }
 
       void drawUI() {
+/*** END OF REFACTORED CODE ***/
          // TODO: Since I currently don't use any other render targets,
          // I may as well set this once before the render loop
@@ -1801 +1805 @@
 
          populateImageFromSDLTexture(uiOverlay, sdlOverlayImage);
-      }
+/*** START OF REFACTORED CODE ***/
+      }
+/*** END OF REFACTORED CODE ***/
 
       void updateUniformBuffer(uint32_t currentImage) {
@@ -1944 +1950 @@
          vkDestroyRenderPass(device, renderPass, nullptr);
 
+/*** START OF REFACTORED CODE ***/
          for (auto imageView : swapChainImageViews) {
             vkDestroyImageView(device, imageView, nullptr);
          }
 
-/*** START OF REFACTORED CODE ***/
          vkDestroySwapchainKHR(device, swapChain, nullptr);
 /*** END OF REFACTORED CODE ***/

vulkan-utils.cpp

@@ -166 +166 @@
    }
 }
+
+VkImageView VulkanUtils::createImageView(VkDevice device, VkImage image, VkFormat format, VkImageAspectFlags aspectFlags) {
+   VkImageViewCreateInfo viewInfo = {};
+   viewInfo.sType = VK_STRUCTURE_TYPE_IMAGE_VIEW_CREATE_INFO;
+   viewInfo.image = image;
+   viewInfo.viewType = VK_IMAGE_VIEW_TYPE_2D;
+   viewInfo.format = format;
+
+   viewInfo.components.r = VK_COMPONENT_SWIZZLE_IDENTITY;
+   viewInfo.components.g = VK_COMPONENT_SWIZZLE_IDENTITY;
+   viewInfo.components.b = VK_COMPONENT_SWIZZLE_IDENTITY;
+   viewInfo.components.a = VK_COMPONENT_SWIZZLE_IDENTITY;
+
+   viewInfo.subresourceRange.aspectMask = aspectFlags;
+   viewInfo.subresourceRange.baseMipLevel = 0;
+   viewInfo.subresourceRange.levelCount = 1;
+   viewInfo.subresourceRange.baseArrayLayer = 0;
+   viewInfo.subresourceRange.layerCount = 1;
+
+   VkImageView imageView;
+   if (vkCreateImageView(device, &viewInfo, nullptr, &imageView) != VK_SUCCESS) {
+      throw runtime_error("failed to create image view!");
+   }
+
+   return imageView;
+}

vulkan-utils.hpp

@@ -43 +43 @@
       static VkPresentModeKHR chooseSwapPresentMode(const vector<VkPresentModeKHR>& availablePresentModes);
       static VkExtent2D chooseSwapExtent(const VkSurfaceCapabilitiesKHR& capabilities, int width, int height);
+      static VkImageView createImageView(VkDevice device, VkImage image, VkFormat format, VkImageAspectFlags aspectFlags);
 };