Fix: debug callback in opengl plugin

with preprocesor macro in readme

Author: imbobz

CMakeLists.txt

@@ -3,6 +3,7 @@ cmake_minimum_required(VERSION 4.0.0)
 project(cae
         VERSION 0.0.0
         DESCRIPTION "Cross-API graphics engine"
+        HOMEPAGE_URL "https://bobis33.github.io/Cross-API-Engine/"
         LANGUAGES C CXX
 )
 

README.md

@@ -83,6 +83,18 @@ This section targets engine developers and contributors.
 - `CAE_CLANG_FORMAT` (default: `OFF`): Enable clang format usage.
 - `CAE_BUILD_DOC` (default: `OFF`): Enable building doxygen documentation.
 
+### Preprocessor macros
+
+- `PLUGINS_DIR`: Path of plugins binaries directory.
+- `CAE_DEBUG`: Defined if build mode is debug.
+- `CAE_DYNAMIC_PLUGINS`: Defined if plugins are build dynamically.
+- `CAE_PLATFORM_WEB`: Defined if platform is Web.
+- `CAE_PLATFORM_ANDROID`: Defined if platform is Android.
+- `CAE_PLATFORM_IOS`: Defined if platform is IOS.
+- `CAE_PLATFORM_MACOS`: Defined if platform is macOS.
+- `CAE_PLATFORM_WINDOWS`: Defined if platform is Windows.
+- `CAE_PLATFORM_LINUX`: Defined if platform is Linux.
+
 ### Testing
 
 Unit tests are based on [Google Test](https://github.com/google/googletest).

cmake/modules/CompileOptions.cmake

@@ -91,7 +91,6 @@ endfunction()
 if (EMSCRIPTEN)
     target_compile_definitions(cae-compile-options INTERFACE
             CAE_PLATFORM_WEB
-            CAE_PLATFORM_EMSCRIPTEN
     )
 elseif (ANDROID)
     target_compile_definitions(cae-compile-options INTERFACE CAE_PLATFORM_ANDROID)

plugins/Renderer/OpenGL/src/context/EGLContext.cpp

@@ -2,6 +2,8 @@
 
 #include "OPGL/Context/EGLContext.hpp"
 
+#include "Utils/Logger.hpp"
+
 #include <stdexcept>
 
 cae::EGLContext_::~EGLContext_()
@@ -85,6 +87,17 @@ void cae::EGLContext_::initialize(const NativeWindowHandle &window)
     {
         throw std::runtime_error("Failed to initialize GLAD");
     }
+#ifdef CAE_DEBUG
+    if (gl.Enable != nullptr)
+    {
+        gl.Enable(GL_DEBUG_OUTPUT);
+
+        gl.DebugMessageCallback([](GLenum source, GLenum type, GLuint id, GLenum severity, GLsizei length,
+                               const GLchar *message, const void *userParam)
+                            { utl::Logger::log("[GL DEBUG] " + std::string(message), utl::LogLevel::WARNING); },
+                            nullptr);
+    }
+#endif
 }
 
 void cae::EGLContext_::swapBuffers()

plugins/Renderer/OpenGL/src/context/NSGLContext.mm

@@ -2,6 +2,8 @@
 
 #include "OPGL/Context/NSGLContext.hpp"
 
+#include "Utils/Logger.hpp"
+
 #import <AppKit/AppKit.h>
 
 cae::NSGLContext::~NSGLContext() {
@@ -33,6 +35,17 @@
     if (!gladLoadGLContext(&gl, nullptr)) {
         throw std::runtime_error("Failed to initialize GLAD");
     }
+#ifdef CAE_DEBUG
+    if (gl.Enable != nullptr)
+    {
+        gl.Enable(GL_DEBUG_OUTPUT);
+
+        gl.DebugMessageCallback([](GLenum source, GLenum type, GLuint id, GLenum severity, GLsizei length,
+                               const GLchar *message, const void *userParam)
+                            { utl::Logger::log("[GL DEBUG] " + std::string(message), utl::LogLevel::WARNING); },
+                            nullptr);
+    }
+#endif
 }
 
 void cae::NSGLContext::swapBuffers() {

plugins/Renderer/OpenGL/src/context/WGLContext.cpp

@@ -143,16 +143,17 @@ void cae::WGLContext::initialize(const NativeWindowHandle &window)
     {
         throw std::runtime_error("Failed to initialize GLAD MX (Windows)");
     }
+#ifdef CAE_DEBUG
     if (gl.Enable != nullptr)
     {
         gl.Enable(GL_DEBUG_OUTPUT);
-#ifdef CAE_DEBUG
         gl.DebugMessageCallback([](GLenum source, GLenum type, GLuint id, GLenum severity, GLsizei length,
                                    const GLchar *message, const void *userParam)
                                 { utl::Logger::log("[GL DEBUG] " + std::string(message), utl::LogLevel::WARNING); },
                                 nullptr);
-#endif
     }
+#endif
+
 }
 
 void cae::WGLContext::swapBuffers()