Update Documentation and remove duplicate includes caused by global headers

src/DeviceLibrary.cpp

@@ -112,6 +112,7 @@ namespace DeviceControl {
   void DeviceLibrary::createLogicalDevice(VkDevice& device) {
     // Describe how many queues we want for a single family (1) here, right now we are solely interested in graphics capabilites,
     // but Compute Shaders, transfer ops, decode and encode operations can also queued with setup! We also assign each queue a priority.
+    // We do this by looping over all the queueFamilies and sorting them by indices to fill the queue at the end!
     QueueFamilyIndices indices = findQueueFamilies(physicalDevice);
 
     std::vector<VkDeviceQueueCreateInfo> queueCreateInfos;
@@ -130,11 +131,10 @@ namespace DeviceControl {
       queueCreateInfos.push_back(queueCreateSingularInfo);
     }
     VkDeviceCreateInfo createDeviceInfo = {};
-    VkPhysicalDeviceFeatures emptyFeatures = {};
     createDeviceInfo.sType = VK_STRUCTURE_TYPE_DEVICE_CREATE_INFO; 
     createDeviceInfo.pQueueCreateInfos = queueCreateInfos.data();
     createDeviceInfo.queueCreateInfoCount = static_cast<uint32_t>(queueCreateInfos.size());
-    createDeviceInfo.pEnabledFeatures = &emptyFeatures;
+    createDeviceInfo.pEnabledFeatures = &deviceFeatures;
     createDeviceInfo.enabledExtensionCount = 0;
 
 

src/debug/VulkanDebugLibs.cpp

@@ -1,14 +1,10 @@
-#include <cstdint>
-#include <stdexcept>
-#include <vulkan/vk_platform.h>
-#define GLFW_INCLUDE_VULKAN
-#include <GLFW/glfw3.h>
-
 #include "../global.h"
 using namespace Debug;
 
+#include <cstdint>
+#include <stdexcept>
+#include <vulkan/vk_platform.h>
 #include <cstring>
-#include <vulkan/vulkan_core.h>
 
 // This is our messenger object! It handles passing along debug messages to the debug callback we will also set.
 VkDebugUtilsMessengerEXT debugMessenger;

src/global.h

@@ -8,6 +8,8 @@
 #include <GLFW/glfw3.h>
 
 namespace Global {
+  // Global variables and includes we are going to use almost everywhere, validation layers hook into everything, and you need to check if they are enabled first,
+  // so that's one obvious global, as well as the glfw includes!
   extern const std::vector<const char*> validationLayers;
   extern const bool enableValidationLayers;
 }

src/main.cpp

@@ -1,8 +1,6 @@
 
+#include "DeviceLibrary.h" // Device Library includes global, redundant to include with it here
 #include "debug/VulkanDebugLibs.h"
-#include "DeviceLibrary.h"
-#include "debug/VulkanDebugLibs.h"
-#include "global.h"
 
 #include <cstdint>
 #include <cstring>