Skip to content

Commit c40f73f

Browse files
committed
[window] Move window and surface code to subfolder in wrapper folder
1 parent 930512e commit c40f73f

File tree

7 files changed

+28
-21
lines changed

7 files changed

+28
-21
lines changed

include/inexor/vulkan-renderer/renderer.hpp

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,14 @@
77
#include "inexor/vulkan-renderer/time_step.hpp"
88
#include "inexor/vulkan-renderer/wrapper/instance.hpp"
99
#include "inexor/vulkan-renderer/wrapper/uniform_buffer.hpp"
10-
#include "inexor/vulkan-renderer/wrapper/window.hpp"
11-
#include "inexor/vulkan-renderer/wrapper/window_surface.hpp"
10+
#include "inexor/vulkan-renderer/wrapper/window/surface.hpp"
11+
#include "inexor/vulkan-renderer/wrapper/window/window.hpp"
1212

1313
namespace inexor::vulkan_renderer {
1414

15+
using wrapper::window::Window;
16+
using wrapper::window::WindowSurface;
17+
1518
class VulkanRenderer {
1619
protected:
1720
std::vector<VkPipelineShaderStageCreateInfo> m_shader_stages;
@@ -24,7 +27,7 @@ class VulkanRenderer {
2427

2528
std::uint32_t m_window_width{0};
2629
std::uint32_t m_window_height{0};
27-
wrapper::Window::Mode m_window_mode{wrapper::Window::Mode::WINDOWED};
30+
Window::Mode m_window_mode{Window::Mode::WINDOWED};
2831

2932
std::string m_window_title;
3033

@@ -34,10 +37,10 @@ class VulkanRenderer {
3437

3538
std::unique_ptr<Camera> m_camera;
3639

37-
std::unique_ptr<wrapper::Window> m_window;
40+
std::unique_ptr<Window> m_window;
3841
std::unique_ptr<wrapper::Instance> m_instance;
3942
std::unique_ptr<wrapper::Device> m_device;
40-
std::unique_ptr<wrapper::WindowSurface> m_surface;
43+
std::unique_ptr<WindowSurface> m_surface;
4144
std::unique_ptr<wrapper::Swapchain> m_swapchain;
4245
std::unique_ptr<ImGUIOverlay> m_imgui_overlay;
4346
std::unique_ptr<RenderGraph> m_render_graph;

include/inexor/vulkan-renderer/wrapper/window_surface.hpp renamed to include/inexor/vulkan-renderer/wrapper/window/surface.hpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
#include <GLFW/glfw3.h>
44
#include <volk.h>
55

6-
namespace inexor::vulkan_renderer::wrapper {
6+
namespace inexor::vulkan_renderer::wrapper::window {
77

88
/// @brief RAII wrapper class for VkSurfaceKHR.
99
class WindowSurface {
@@ -29,4 +29,4 @@ class WindowSurface {
2929
}
3030
};
3131

32-
} // namespace inexor::vulkan_renderer::wrapper
32+
} // namespace inexor::vulkan_renderer::wrapper::window

include/inexor/vulkan-renderer/wrapper/window.hpp renamed to include/inexor/vulkan-renderer/wrapper/window/window.hpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
#include <cstdint>
66
#include <string>
77

8-
namespace inexor::vulkan_renderer::wrapper {
8+
namespace inexor::vulkan_renderer::wrapper::window {
99

1010
/// @brief RAII wrapper class for GLFW windows.
1111
class Window {
@@ -93,4 +93,4 @@ class Window {
9393
}
9494
};
9595

96-
} // namespace inexor::vulkan_renderer::wrapper
96+
} // namespace inexor::vulkan_renderer::wrapper::window

src/CMakeLists.txt

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,9 @@ set(INEXOR_SOURCE_FILES
3838
vulkan-renderer/wrapper/shader.cpp
3939
vulkan-renderer/wrapper/swapchain.cpp
4040
vulkan-renderer/wrapper/uniform_buffer.cpp
41-
vulkan-renderer/wrapper/window.cpp
42-
vulkan-renderer/wrapper/window_surface.cpp
41+
42+
vulkan-renderer/wrapper/window/window.cpp
43+
vulkan-renderer/wrapper/window/surface.cpp
4344

4445
vulkan-renderer/world/collision.cpp
4546
vulkan-renderer/world/collision_query.cpp

src/vulkan-renderer/application.cpp

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,8 @@ void Application::load_toml_configuration_file(const std::string &file_name) {
4444
const auto &configuration_title = toml::find<std::string>(renderer_configuration, "title");
4545
spdlog::trace("Title: {}", configuration_title);
4646

47-
using WindowMode = ::inexor::vulkan_renderer::wrapper::Window::Mode;
47+
using WindowMode = wrapper::window::Window::Mode;
48+
4849
const auto &wmodestr = toml::find<std::string>(renderer_configuration, "application", "window", "mode");
4950
if (wmodestr == "windowed") {
5051
m_window_mode = WindowMode::WINDOWED;
@@ -336,8 +337,10 @@ Application::Application(int argc, char **argv) {
336337

337338
spdlog::trace("Creating Vulkan instance");
338339

339-
m_window =
340-
std::make_unique<wrapper::Window>(m_window_title, m_window_width, m_window_height, true, true, m_window_mode);
340+
using wrapper::window::Window;
341+
using wrapper::window::WindowSurface;
342+
343+
m_window = std::make_unique<Window>(m_window_title, m_window_width, m_window_height, true, true, m_window_mode);
341344

342345
m_instance = std::make_unique<wrapper::Instance>(
343346
APP_NAME, ENGINE_NAME, VK_MAKE_API_VERSION(0, APP_VERSION[0], APP_VERSION[1], APP_VERSION[2]),
@@ -346,7 +349,7 @@ Application::Application(int argc, char **argv) {
346349

347350
m_input = std::make_unique<input::Input>();
348351

349-
m_surface = std::make_unique<wrapper::WindowSurface>(m_instance->instance(), m_window->get());
352+
m_surface = std::make_unique<WindowSurface>(m_instance->instance(), m_window->get());
350353

351354
setup_window_and_input_callbacks();
352355

src/vulkan-renderer/wrapper/window_surface.cpp renamed to src/vulkan-renderer/wrapper/window/surface.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
#include "inexor/vulkan-renderer/wrapper/window_surface.hpp"
1+
#include "inexor/vulkan-renderer/wrapper/window/surface.hpp"
22

33
#include "inexor/vulkan-renderer/exception.hpp"
44

@@ -7,7 +7,7 @@
77
#include <cassert>
88
#include <utility>
99

10-
namespace inexor::vulkan_renderer::wrapper {
10+
namespace inexor::vulkan_renderer::wrapper::window {
1111

1212
WindowSurface::WindowSurface(const VkInstance instance, GLFWwindow *window) : m_instance(instance) {
1313
assert(instance);
@@ -29,4 +29,4 @@ WindowSurface::~WindowSurface() {
2929
vkDestroySurfaceKHR(m_instance, m_surface, nullptr);
3030
}
3131

32-
} // namespace inexor::vulkan_renderer::wrapper
32+
} // namespace inexor::vulkan_renderer::wrapper::window

src/vulkan-renderer/wrapper/window.cpp renamed to src/vulkan-renderer/wrapper/window/window.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
#include "inexor/vulkan-renderer/wrapper/window.hpp"
1+
#include "inexor/vulkan-renderer/wrapper/window/window.hpp"
22

33
#include <GLFW/glfw3.h>
44
#include <spdlog/spdlog.h>
55

66
#include <cassert>
77
#include <stdexcept>
88

9-
namespace inexor::vulkan_renderer::wrapper {
9+
namespace inexor::vulkan_renderer::wrapper::window {
1010

1111
Window::Window(const std::string &title, const std::uint32_t width, const std::uint32_t height, const bool visible,
1212
const bool resizable, const Mode mode)
@@ -99,4 +99,4 @@ bool Window::should_close() {
9999
return glfwWindowShouldClose(m_window) == GLFW_TRUE;
100100
}
101101

102-
} // namespace inexor::vulkan_renderer::wrapper
102+
} // namespace inexor::vulkan_renderer::wrapper::window

0 commit comments

Comments
 (0)