fix: relative mouse mode resets

CMakeLists.txt

@@ -1,6 +1,8 @@
 cmake_minimum_required(VERSION 3.16)
 project(CodingGame LANGUAGES C CXX)
 
+set(ENGINE_TARGET engine)
+
 # ---------- Build-type defaults (only affects single-config generators like Ninja/Make) ----------
 if(NOT CMAKE_CONFIGURATION_TYPES AND NOT CMAKE_BUILD_TYPE)
   set(CMAKE_BUILD_TYPE Release CACHE STRING "Choose the type of build." FORCE)
@@ -12,7 +14,7 @@ if (UNIX)
   FetchContent_Declare(
     glm
     GIT_REPOSITORY https://github.com/g-truc/glm.git
-    GIT_TAG        bf71a834948186f4097caa076cd2663c69a10e1e # refs/tags/1.0.1
+    GIT_TAG        a532f5b1cf27d6a3c099437e6959cf7e398a0a67 # refs/tags/1.0.2
   )
   FetchContent_MakeAvailable(glm)
 
@@ -34,99 +36,5 @@ elseif (MSVC) # vcpkg
   find_package(EnTT CONFIG REQUIRED)
 endif()
 
-add_executable(CodingGame
+add_subdirectory(engine)
-  src/IO/parser.cpp
+add_subdirectory(sandbox)
-  src/IO/file_manager.cpp
-
-  src/window/window.cpp
-
-  src/components/batch.cpp
-
-  src/renderer/debug.cpp
-  src/renderer/mesh.cpp
-  src/renderer/shader.cpp
-  src/renderer/texture.cpp
-  src/renderer/wavefront.cpp
-  src/renderer/engine.cpp
-  src/renderer/renderer.cpp
-
-  src/main.cpp
-)
-
-set_property(TARGET CodingGame PROPERTY CXX_STANDARD 17)
-set_property(TARGET CodingGame PROPERTY CXX_STANDARD_REQUIRED ON)
-
-file(COPY src/shaders DESTINATION ${CMAKE_BINARY_DIR}/)
-
-target_include_directories(CodingGame PRIVATE
-  ${CMAKE_SOURCE_DIR}/include
-  ${CMAKE_SOURCE_DIR}/contrib
-)
-target_link_libraries(CodingGame PRIVATE
-  SDL3::SDL3
-  OpenGL::GL
-  GLEW::GLEW
-  glm::glm
-  EnTT::EnTT
-)
-
-# ---------- Visibility (helps optimizer & smaller binaries on Release) ----------
-# Only affects non-Windows compilers
-set_target_properties(CodingGame PROPERTIES
-  CXX_VISIBILITY_PRESET hidden
-  VISIBILITY_INLINES_HIDDEN YES
-)
-
-# ---------- Per-config flags ----------
-# Debug flags (your original intent, kept)
-if (MSVC)
-  target_compile_options(CodingGame PRIVATE $<$<CONFIG:Debug>:/Zi>)
-  target_link_options(CodingGame    PRIVATE $<$<CONFIG:Debug>:/DEBUG:FULL>)
-else()
-  target_compile_options(CodingGame PRIVATE $<$<CONFIG:Debug>:-ggdb>)
-endif()
-
-# Release flags
-if (MSVC)
-  # /O2: optimize speed, /GL: whole program opt (LTCG), /DNDEBUG: disable asserts
-  target_compile_options(CodingGame PRIVATE
-    $<$<CONFIG:Release>:/O2>
-    $<$<CONFIG:Release>:/DNDEBUG>
-    $<$<CONFIG:RelWithDebInfo>:/O2>
-  )
-  # Link-time codegen & extra linker opts for smaller/faster binaries
-  target_link_options(CodingGame PRIVATE
-    $<$<CONFIG:Release>:/LTCG /OPT:ICF /OPT:REF>
-    $<$<CONFIG:RelWithDebInfo>:/LTCG /OPT:ICF /OPT:REF>
-  )
-else()
-  # GCC/Clang
-  # -O3 for max opts, -ffast-math optional but can be risky; we keep it OFF by default.
-  option(CODINGGAME_USE_MARCH_NATIVE "Enable -march=native on Release for this machine" ON)
-  target_compile_options(CodingGame PRIVATE
-    $<$<CONFIG:Release>:-O3>
-    $<$<AND:$<CONFIG:Release>,$<BOOL:${CODINGGAME_USE_MARCH_NATIVE}>>:-march=native>
-    $<$<CONFIG:Release>:-DNDEBUG>
-    $<$<CONFIG:RelWithDebInfo>:-O3 -g>
-  )
-  # Linker: enable LTO when available; optionally strip symbols on non-Apple
-  include(CheckIPOSupported)  # IPO == LTO in CMake terms
-  check_ipo_supported(RESULT ipo_supported OUTPUT ipo_msg)
-  if(ipo_supported)
-    set_property(TARGET CodingGame PROPERTY INTERPROCEDURAL_OPTIMIZATION_RELEASE TRUE)
-    set_property(TARGET CodingGame PROPERTY INTERPROCEDURAL_OPTIMIZATION_RELWITHDEBINFO TRUE)
-  endif()
-
-  if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU" AND NOT APPLE)
-    # -s strips symbols at link stage (use only for pure Release)
-    target_link_options(CodingGame PRIVATE $<$<CONFIG:Release>:-s>)
-  endif()
-endif()
-
-# ---------- Windows: copy runtime DLLs ----------
-if (WIN32)
-  add_custom_command(TARGET CodingGame POST_BUILD
-    COMMAND ${CMAKE_COMMAND} -E copy_if_different
-      $<TARGET_RUNTIME_DLLS:CodingGame> $<TARGET_FILE_DIR:CodingGame>
-    COMMAND_EXPAND_LISTS)
-endif()

engine/CMakeLists.txt

@@ -0,0 +1,89 @@
+add_library(${ENGINE_TARGET} STATIC
+  src/IO/parser.cpp
+  src/IO/file_manager.cpp
+
+  src/window/window.cpp
+
+  src/components/batch.cpp
+
+  src/renderer/debug.cpp
+  src/renderer/mesh.cpp
+  src/renderer/shader.cpp
+  src/renderer/texture.cpp
+  src/renderer/wavefront.cpp
+  src/renderer/core.cpp
+  src/renderer/renderer.cpp
+
+  # src/main.cpp
+)
+
+set_target_properties(${ENGINE_TARGET} PROPERTIES
+  CXX_STANDARD 17
+  CXX_STANDARD_REQUIRED ON
+  CXX_VISIBILITY_PRESET hidden
+  VISIBILITY_INLINES_HIDDEN YES
+)
+
+target_include_directories(${ENGINE_TARGET} PUBLIC
+  ${CMAKE_CURRENT_SOURCE_DIR}/include
+  ${CMAKE_CURRENT_SOURCE_DIR}/contrib
+)
+target_link_libraries(${ENGINE_TARGET} PUBLIC
+  SDL3::SDL3
+  OpenGL::GL
+  GLEW::GLEW
+  glm::glm
+  EnTT::EnTT
+)
+
+if (MSVC)
+  target_compile_options(${ENGINE_TARGET} PRIVATE $<$<CONFIG:Debug>:/Zi>)
+  target_link_options(${ENGINE_TARGET}    PRIVATE $<$<CONFIG:Debug>:/DEBUG:FULL>)
+else()
+  target_compile_options(${ENGINE_TARGET} PRIVATE $<$<CONFIG:Debug>:-ggdb>)
+endif()
+
+# Release flags
+if (MSVC)
+  # /O2: optimize speed, /GL: whole program opt (LTCG), /DNDEBUG: disable asserts
+  target_compile_options(${ENGINE_TARGET} PRIVATE
+    $<$<CONFIG:Release>:/O2>
+    $<$<CONFIG:Release>:/DNDEBUG>
+    $<$<CONFIG:RelWithDebInfo>:/O2>
+  )
+  # Link-time codegen & extra linker opts for smaller/faster binaries
+  target_link_options(${ENGINE_TARGET} PRIVATE
+    $<$<CONFIG:Release>:/LTCG /OPT:ICF /OPT:REF>
+    $<$<CONFIG:RelWithDebInfo>:/LTCG /OPT:ICF /OPT:REF>
+  )
+else()
+  # GCC/Clang
+  # -O3 for max opts, -ffast-math optional but can be risky; we keep it OFF by default.
+  option(CODINGGAME_USE_MARCH_NATIVE "Enable -march=native on Release for this machine" ON)
+  target_compile_options(${ENGINE_TARGET} PRIVATE
+    $<$<CONFIG:Release>:-O3>
+    $<$<AND:$<CONFIG:Release>,$<BOOL:${CODINGGAME_USE_MARCH_NATIVE}>>:-march=native>
+    $<$<CONFIG:Release>:-DNDEBUG>
+    $<$<CONFIG:RelWithDebInfo>:-O3 -g>
+  )
+  # Linker: enable LTO when available; optionally strip symbols on non-Apple
+  include(CheckIPOSupported)  # IPO == LTO in CMake terms
+  check_ipo_supported(RESULT ipo_supported OUTPUT ipo_msg)
+  if(ipo_supported)
+    set_property(TARGET ${ENGINE_TARGET} PROPERTY INTERPROCEDURAL_OPTIMIZATION_RELEASE TRUE)
+    set_property(TARGET ${ENGINE_TARGET} PROPERTY INTERPROCEDURAL_OPTIMIZATION_RELWITHDEBINFO TRUE)
+  endif()
+
+  if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU" AND NOT APPLE)
+    # -s strips symbols at link stage (use only for pure Release)
+    target_link_options(${ENGINE_TARGET} PRIVATE $<$<CONFIG:Release>:-s>)
+  endif()
+endif()
+
+# ---------- Windows: copy runtime DLLs ----------
+if (WIN32)
+  add_custom_command(TARGET ${ENGINE_TARGET} POST_BUILD
+    COMMAND ${CMAKE_COMMAND} -E copy_if_different
+      $<TARGET_RUNTIME_DLLS:${ENGINE_TARGET}> $<TARGET_FILE_DIR:${ENGINE_TARGET}>
+    COMMAND_EXPAND_LISTS)
+endif()

engine/include/engine/app/app.h

@@ -1,6 +1,8 @@
 #ifndef APPLICATION_H_
 #define APPLICATION_H_
 
+#include "engine/window/events/window.h"
+
 class IApplication {
 public:
     virtual ~IApplication() = default;

engine/include/engine/components/batch.h

@@ -1,7 +1,7 @@
 #ifndef COMPONENT_BATCH_H_
 #define COMPONENT_BATCH_H_
 
-#include "renderer/renderer.h"
+#include "engine/renderer/renderer.h"
 
 // requires mesh component
 struct batch {

engine/include/engine/components/light.h

@@ -2,7 +2,7 @@
 #define COMPONENTS_LIGHT_H_
 
 #include <glm/glm.hpp>
-#include "renderer/renderer.h"
+#include "engine/renderer/renderer.h"
 
 struct light {
     friend class Renderer;

engine/include/engine/components/mesh.h

@@ -2,7 +2,7 @@
 #define COMPONENTS_MESH_H_
 
 #include <memory>
-#include "renderer/wavefront.h"
+#include "engine/renderer/wavefront.h"
 
 struct mesh {
     std::shared_ptr<Object> object;

engine/include/engine/engine.h

@@ -0,0 +1,19 @@
+#ifndef WIN32
+#define GLEW_STATIC
+#endif
+
+#include <memory>
+
+#ifdef WIN32
+#include <corecrt_math_defines.h>
+#endif
+
+#include "engine/app/app.h"
+#include "engine/renderer/core.h"
+
+extern IApplication* CreateApplication();
+
+int main() {
+    Engine::Run(std::unique_ptr<IApplication>(CreateApplication()));
+    return 0;
+}

engine/include/engine/renderer/basics.h

@@ -3,7 +3,7 @@
 
 #include <glm/glm.hpp>
 
-#include "renderer/mesh.h"
+#include "engine/renderer/mesh.h"
 
 class Vertex {
     friend class Mesh;

engine/include/engine/renderer/core.h

@@ -4,10 +4,10 @@
 #include <memory>
 #include <glm/glm.hpp>
 
-#include "window/window.h"
+#include "engine/window/window.h"
-#include "window/events/window.h"
+#include "engine/window/events/window.h"
 
-#include "app/app.h"
+#include "engine/app/app.h"
 
 class Engine {
 public:

engine/include/engine/renderer/material.h

@@ -4,7 +4,7 @@
 #include <glm/glm.hpp>
 #include <memory>
 
-#include "texture.h"
+#include "engine/renderer/texture.h"
 
 class Material {
 private:

engine/include/engine/renderer/mesh.h

@@ -5,7 +5,7 @@
 #include <string>
 #include <GL/glew.h>
 
-#include "renderer/basics.h"
+#include "engine/renderer/basics.h"
 
 class Mesh {
 public: // TODO: abstract away

engine/include/engine/renderer/renderer.h

@@ -4,7 +4,7 @@
 #include <glm/glm.hpp>
 #include <entt/entity/registry.hpp>
 
-#include "renderer/shader.h"
+#include "engine/renderer/shader.h"
 
 // TODO: make static or singleton
 class Renderer {

engine/include/engine/renderer/wavefront.h

@@ -7,10 +7,10 @@
 #include <glm/glm.hpp>
 #include <memory>
 
-#include "shader.h"
+#include "engine/renderer/shader.h"
-#include "renderer/renderer.h"
+#include "engine/renderer/renderer.h"
-#include "renderer/material.h"
+#include "engine/renderer/material.h"
-#include "renderer/mesh.h"
+#include "engine/renderer/mesh.h"
 
 enum ObjElement { OHASH, MTLLIB, USEMTL, O, V, VN, VT, F, OUNKNOWN };
 enum MtlElement { MHASH, NEWMTL, NS, KA, KS, KD, NI, D, ILLUM, MAP_KD, MAP_KA, MUNKNOWN };

engine/include/engine/window/window.h

@@ -3,7 +3,7 @@
 #include <SDL3/SDL.h>
 #include <memory>
 
-#include "event.h"
+#include "engine/window/event.h"
 
 #define ENGINE_GL_MAJOR_VERSION 4
 #define ENGINE_GL_MINOR_VERSION 6

engine/src/IO/file_manager.cpp

@@ -1,4 +1,4 @@
-#include "IO/file_manager.h"
+#include "engine/IO/file_manager.h"
 
 #include <fstream>
 #include <iostream>

engine/src/IO/parser.cpp

@@ -2,7 +2,7 @@
 #include <cstdlib>    // for strtof (fallback)
 #include <cstring>
 
-#include "IO/parser.h"
+#include "engine/IO/parser.h"
 
 // Skip whitespace
 void Parser::SkipSpaces() {

engine/src/components/batch.cpp

@@ -1,6 +1,6 @@
 #include <GL/glew.h>
 
-#include "components/batch.h"
+#include "engine/components/batch.h"
 
 unsigned int batch::LastID = 0;
 

engine/src/renderer/core.cpp

@@ -1,9 +1,9 @@
 #include <memory>
 
-#include "renderer/engine.h"
+#include "engine/renderer/core.h"
-#include "window/event.h"
+#include "engine/window/event.h"
 
-#include "renderer/wavefront.h"
+#include "engine/renderer/wavefront.h"
 
 std::unique_ptr<IApplication> Engine::s_app = nullptr;
 std::shared_ptr<Window> Engine::s_window = nullptr;

engine/src/renderer/debug.cpp

@@ -1,4 +1,4 @@
-#include "renderer/debug.h"
+#include "engine/renderer/debug.h"
 
 #include <iostream>
 

engine/src/renderer/mesh.cpp

@@ -1,6 +1,6 @@
 #include <cstddef>
 
-#include "renderer/mesh.h"
+#include "engine/renderer/mesh.h"
 
 Mesh::Mesh() {
     m_vao = 0;

engine/src/renderer/renderer.cpp

@@ -9,15 +9,15 @@
 #define GLM_ENABLE_EXPERIMENTAL
 #include <glm/gtx/euler_angles.hpp>
 
-#include "renderer/renderer.h"
+#include "engine/renderer/renderer.h"
-#include "window/window.h"
+#include "engine/window/window.h"
-#include "IO/file_manager.h"
+#include "engine/IO/file_manager.h"
 
-#include "components/transform.h"
+#include "engine/components/transform.h"
-#include "components/camera.h"
+#include "engine/components/camera.h"
-#include "components/light.h"
+#include "engine/components/light.h"
-#include "components/mesh.h"
+#include "engine/components/mesh.h"
-#include "components/batch.h"
+#include "engine/components/batch.h"
 
 Renderer::Renderer(entt::registry& registry) : m_registry(registry)
 {
@@ -29,13 +29,13 @@ Renderer::Renderer(entt::registry& registry) : m_registry(registry)
     );
 
     m_shader.init(
-        FileManager::read("./src/shaders/main.vs"),
+        FileManager::read("./engine/src/shaders/main.vs"),
-        FileManager::read("./src/shaders/pbr.fs")
+        FileManager::read("./engine/src/shaders/pbr.fs")
     );
 
     m_depthShader.init(
-        FileManager::read("./src/shaders/depth.vs"),
+        FileManager::read("./engine/src/shaders/depth.vs"),
-        FileManager::read("./src/shaders/depth.fs")
+        FileManager::read("./engine/src/shaders/depth.fs")
     );
 
     m_model = glm::mat4(1.f);

engine/src/renderer/shader.cpp

@@ -1,6 +1,6 @@
 #include <iostream>
 #include <GL/glew.h>
-#include "renderer/shader.h"
+#include "engine/renderer/shader.h"
 
 Shader::Shader()
 {

engine/src/renderer/texture.cpp

@@ -2,7 +2,7 @@
 #include <memory>
 
 #include <GL/glew.h>
-#include "renderer/texture.h"
+#include "engine/renderer/texture.h"
 
 #define STB_IMAGE_IMPLEMENTATION
 #include "stb_image.h"

engine/src/renderer/wavefront.cpp

@@ -5,9 +5,9 @@
 #include <filesystem>
 #include <GL/glew.h>
 
-#include "IO/parser.h"
+#include "engine/IO/parser.h"
-#include "renderer/mesh.h"
+#include "engine/renderer/mesh.h"
-#include "renderer/wavefront.h"
+#include "engine/renderer/wavefront.h"
 
 #define DEFAULT_MATERIAL_NAME "default"
 

engine/src/window/window.cpp

@@ -1,12 +1,12 @@
 #include <SDL3/SDL.h>
 
-#include "window/window.h"
+#include "engine/window/window.h"
-#include "window/events/window.h"
+#include "engine/window/events/window.h"
 
 #include <iostream>
 #include <GL/glew.h>
 
-#include "renderer/debug.h"
+#include "engine/renderer/debug.h"
 
 std::shared_ptr<Window> Window::s_instance = nullptr;
 
@@ -31,6 +31,8 @@ Window::Window(const char* title, int width, int height) {
     }
 
     SDL_SetWindowRelativeMouseMode(m_handle, true);
+    SDL_Rect boundaries = {0, 0, m_width, m_height};
+    SDL_SetWindowMouseRect(m_handle, &boundaries);
 
     m_context = SDL_GL_CreateContext(m_handle);
 
@@ -121,6 +123,9 @@ void Window::ProcessEvents() {
                         width,
                         height);
                     Dispatch(WindowResized{ m_width, m_height });
+                    SDL_SetWindowRelativeMouseMode(m_handle, true);
+                    SDL_Rect boundaries = {0, 0, m_width, m_height};
+                    SDL_SetWindowMouseRect(m_handle, &boundaries);
                 }
                 break;
             default: break;

sandbox/CMakeLists.txt

@@ -0,0 +1,10 @@
+set(SANDBOX_TARGET sandbox)
+
+add_executable(${SANDBOX_TARGET} src/main.cpp)
+
+set_target_properties(${SANDBOX_TARGET} PROPERTIES
+  CXX_STANDARD 17
+  CXX_STANDARD_REQUIRED ON
+)
+
+target_link_libraries(${SANDBOX_TARGET} PRIVATE ${ENGINE_TARGET})

sandbox/src/main.cpp

@@ -1,30 +1,21 @@
-#ifndef WIN32
-#define GLEW_STATIC
-#endif
-
 #include <iostream>
-#include <memory>
 
-#ifdef WIN32
-#include <corecrt_math_defines.h>
-#endif
 #include <glm/glm.hpp>
-#include <glm/ext/matrix_clip_space.hpp>
+#include <glm/gtc/constants.hpp>
-#include <glm/ext/matrix_transform.hpp>
 
-#include "renderer/shader.h"
+#include "engine/renderer/wavefront.h"
-#include "renderer/wavefront.h"
+#include "engine/renderer/renderer.h"
-#include "renderer/engine.h"
-#include "renderer/renderer.h"
 
-#include "IO/file_manager.h"
+#include "engine/app/app.h"
 
-#include "components/transform.h"
+#include "engine/components/transform.h"
-#include "components/light.h"
+#include "engine/components/light.h"
-#include "components/camera.h"
+#include "engine/components/camera.h"
-#include "components/mesh.h"
+#include "engine/components/mesh.h"
-#include "components/rotate.h"
+#include "engine/components/rotate.h"
-#include "components/batch.h"
+#include "engine/components/batch.h"
+
+#include "engine/engine.h"
 
 class Game : public IApplication {
 public:
@@ -232,7 +223,6 @@ private:
     Uint64 m_currentTicks;
 };
 
-int main() {
+IApplication* CreateApplication() {
-    Engine::Run(std::make_unique<Game>());
+    return new Game();
-    return 0;
+}
-}