Changeset 7865c5b in opengl-game for vulkan-game.cpp

Timestamp:

Mar 17, 2021, 12:50:49 AM (4 years ago)

Author:

Dmitry Portnoy <dportnoy@…>

Branches:

feature/imgui-sdl

Children:

aa7e5f0

Parents:

85b5fec

Message:

Rename surface to vulkanSurface and add an initializer list to the VulkanGame constructor

File:

• vulkan-game.cpp (modified) (14 diffs)

vulkan-game.cpp

@@ -50 +50 @@
 }
 
-VulkanGame::VulkanGame() {
-   // TODO: Double-check whether initialization should happen in the header, where the variables are declared, or here
-   // Also, decide whether to use this-> for all instance variables, or only when necessary
-
-   debugMessenger = VK_NULL_HANDLE;
-
-   gui = nullptr;
-   window = nullptr;
-
-   swapChainPresentMode = VK_PRESENT_MODE_MAX_ENUM_KHR;
-   swapChainMinImageCount = 0;
-   currentFrame = 0;
-   imageIndex = 0;
-   shouldRecreateSwapChain = false;
-
+VulkanGame::VulkanGame()
+                     : swapChainImageCount(0)
+                      , swapChainMinImageCount(0)
+                     , swapChainSurfaceFormat({})
+                     , swapChainPresentMode(VK_PRESENT_MODE_MAX_ENUM_KHR)
+                     , swapChainExtent{ 0, 0 }
+                     , swapChain(VK_NULL_HANDLE)
+                     , vulkanSurface(VK_NULL_HANDLE)
+                     , sdlVersion({ 0, 0, 0 })
+                     , instance(VK_NULL_HANDLE)
+                     , physicalDevice(VK_NULL_HANDLE)
+                     , device(VK_NULL_HANDLE)
+                     , debugMessenger(VK_NULL_HANDLE)
+                     , resourceCommandPool(VK_NULL_HANDLE)
+                     , renderPass(VK_NULL_HANDLE)
+                     , graphicsQueue(VK_NULL_HANDLE)
+                     , presentQueue(VK_NULL_HANDLE)
+                     , depthImage({})
+                     , shouldRecreateSwapChain(false)
+                     , frameCount(0)
+                     , currentFrame()
+                     , imageIndex(0)
+                     , fpsStartTime(0.0f)
+                     , curTime(0.0f)
+                     , done(false)
+                     , currentRenderScreenFn(nullptr)
+                     , gui(nullptr)
+                     , window(nullptr)
+                     , score(0)
+                     , fps(0.0f) {
    object_VP_mats = {};
    ship_VP_mats = {};
@@ -70 +85 @@
    laser_VP_mats = {};
    explosion_UBO = {};
-
-   score = 0;
-   fps = 0.0f;
 }
 
@@ -205 +217 @@
 
    // TODO: Maybe call this once and save the results since it's also called when creating the logical device
-   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, surface);
+   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, vulkanSurface);
 
    ImGui_ImplSDL2_InitForVulkan(window);
@@ -1149 +1161 @@
 
    vkDestroyDevice(device, nullptr);
-   vkDestroySurfaceKHR(instance, surface, nullptr);
+   vkDestroySurfaceKHR(instance, vulkanSurface, nullptr);
 
    if (ENABLE_VALIDATION_LAYERS) {
@@ -1244 +1256 @@
 
 void VulkanGame::createVulkanSurface() {
-   if (gui->createVulkanSurface(instance, &surface) == RTWO_ERROR) {
+   if (gui->createVulkanSurface(instance, &vulkanSurface) == RTWO_ERROR) {
       throw runtime_error("failed to create window surface!");
    }
@@ -1290 +1302 @@
    }
 
-   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, surface);
+   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, vulkanSurface);
    bool extensionsSupported = VulkanUtils::checkDeviceExtensionSupport(physicalDevice, deviceExtensions);
    bool swapChainAdequate = false;
 
    if (extensionsSupported) {
-      vector<VkSurfaceFormatKHR> formats = VulkanUtils::querySwapChainFormats(physicalDevice, surface);
-      vector<VkPresentModeKHR> presentModes = VulkanUtils::querySwapChainPresentModes(physicalDevice, surface);
+      vector<VkSurfaceFormatKHR> formats = VulkanUtils::querySwapChainFormats(physicalDevice, vulkanSurface);
+      vector<VkPresentModeKHR> presentModes = VulkanUtils::querySwapChainPresentModes(physicalDevice, vulkanSurface);
 
       swapChainAdequate = !formats.empty() && !presentModes.empty();
@@ -1309 +1321 @@
 void VulkanGame::createLogicalDevice(const vector<const char*>& validationLayers,
       const vector<const char*>& deviceExtensions) {
-   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, surface);
+   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, vulkanSurface);
 
    if (!indices.isComplete()) {
@@ -1364 +1376 @@
 
 void VulkanGame::chooseSwapChainProperties() {
-   vector<VkSurfaceFormatKHR> availableFormats = VulkanUtils::querySwapChainFormats(physicalDevice, surface);
-   vector<VkPresentModeKHR> availablePresentModes = VulkanUtils::querySwapChainPresentModes(physicalDevice, surface);
+   vector<VkSurfaceFormatKHR> availableFormats = VulkanUtils::querySwapChainFormats(physicalDevice, vulkanSurface);
+   vector<VkPresentModeKHR> availablePresentModes = VulkanUtils::querySwapChainPresentModes(physicalDevice, vulkanSurface);
 
    swapChainSurfaceFormat = VulkanUtils::chooseSwapSurfaceFormat(availableFormats,
@@ -1380 +1392 @@
    cout << "[vulkan] Selected PresentMode = " << swapChainPresentMode << endl;
 
-   VkSurfaceCapabilitiesKHR capabilities = VulkanUtils::querySwapChainCapabilities(physicalDevice, surface);
+   VkSurfaceCapabilitiesKHR capabilities = VulkanUtils::querySwapChainCapabilities(physicalDevice, vulkanSurface);
 
    if (swapChainPresentMode == VK_PRESENT_MODE_MAILBOX_KHR) {
@@ -1400 +1412 @@
 
 void VulkanGame::createSwapChain() {
-   VkSurfaceCapabilitiesKHR capabilities = VulkanUtils::querySwapChainCapabilities(physicalDevice, surface);
+   VkSurfaceCapabilitiesKHR capabilities = VulkanUtils::querySwapChainCapabilities(physicalDevice, vulkanSurface);
 
    swapChainExtent = VulkanUtils::chooseSwapExtent(capabilities, gui->getWindowWidth(), gui->getWindowHeight());
@@ -1406 +1418 @@
    VkSwapchainCreateInfoKHR createInfo = {};
    createInfo.sType = VK_STRUCTURE_TYPE_SWAPCHAIN_CREATE_INFO_KHR;
-   createInfo.surface = surface;
+   createInfo.surface = vulkanSurface;
    createInfo.minImageCount = swapChainMinImageCount;
    createInfo.imageFormat = swapChainSurfaceFormat.format;
@@ -1415 +1427 @@
 
    // TODO: Maybe save this result so I don't have to recalculate it every time
-   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, surface);
+   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, vulkanSurface);
    uint32_t queueFamilyIndices[] = { indices.graphicsFamily.value(), indices.presentFamily.value() };
 
@@ -1525 +1537 @@
 
 void VulkanGame::createResourceCommandPool() {
-   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, surface);
+   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, vulkanSurface);
 
    VkCommandPoolCreateInfo poolInfo = {};
@@ -1540 +1552 @@
    commandPools.resize(swapChainImageCount);
 
-   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, surface);
+   QueueFamilyIndices indices = VulkanUtils::findQueueFamilies(physicalDevice, vulkanSurface);
 
    for (size_t i = 0; i < swapChainImageCount; i++) {