replace OpenGL depencencies with GLAD

add GL resolver, load GLAD
remove GLEW, adjust sdl-test-ui

Author: mbaetgen-wup

CMakeLists.txt

@@ -165,36 +165,12 @@ else()
 
     if(ENABLE_GLES)
         message(STATUS "Building for OpenGL Embedded Profile")
-        if(NOT CMAKE_SYSTEM_NAME STREQUAL Linux
-                AND NOT CMAKE_SYSTEM_NAME STREQUAL Android)
-            message(FATAL_ERROR "OpenGL ES 3 support is currently only available for Linux platforms. You're building for ${CMAKE_SYSTEM_NAME}.")
-        endif()
-
-        # We use a local find script for OpenGL::GLES3 until the proposed changes are merged upstream.
-        list(APPEND CMAKE_MODULE_PATH "${PROJECTM_SOURCE_DIR}/cmake/gles")
-        find_package(OpenGL REQUIRED COMPONENTS GLES3)
-        if(NOT TARGET OpenGL::GLES3)
-            message(FATAL_ERROR "No suitable GLES3 library was found.")
-        endif()
-
-        set(PROJECTM_OPENGL_LIBRARIES OpenGL::GLES3)
         set(USE_GLES ON)
     else()
         message(STATUS "Building for OpenGL Core Profile")
-        find_package(OpenGL REQUIRED)
-        set(PROJECTM_OPENGL_LIBRARIES OpenGL::GL)
-        # GLX is required by SOIL2 on platforms with the X Window System (e.g. most Linux distributions)
-        if(TARGET OpenGL::GLX)
-            list(APPEND PROJECTM_OPENGL_LIBRARIES OpenGL::GLX)
-        endif()
-        if(CMAKE_SYSTEM_NAME STREQUAL "Windows")
-            find_package(GLEW REQUIRED)
-            # Prefer shared, but check for static lib if shared is not available.
-            if(TARGET GLEW::glew)
-                list(APPEND PROJECTM_OPENGL_LIBRARIES GLEW::glew)
-            elseif(TARGET GLEW::glew_s)
-                list(APPEND PROJECTM_OPENGL_LIBRARIES GLEW::glew_s)
-            endif()
+        if(WIN32)
+            find_package(OpenGL REQUIRED)
+            set(PROJECTM_OPENGL_LIBRARIES OpenGL::GL)
         endif()
     endif()
 endif()

src/api/include/projectM-4/core.h

@@ -38,12 +38,31 @@ extern "C" {
  * If this function returns NULL, in most cases the OpenGL context is not initialized, not made
  * current or insufficient to render projectM visuals.
  *
+ * Note: The OpenGL resolver is initialized on the first call to either projectm_create() or projectm_create_with_opengl_load_proc().
+ * All projectM instances share the same resolver.
+ *
  * @return A projectM handle for the newly created instance that must be used in subsequent API calls.
  *         NULL if the instance could not be created successfully.
  * @since 4.0.0
  */
 PROJECTM_EXPORT projectm_handle projectm_create();
 
+/**
+ * @brief Creates a new projectM instance using the given function to resolve GL api functions.
+ *
+ * The load_proc function accepts a function name and a user data pointer.
+ * If this function returns NULL, in most cases the OpenGL context is not initialized, not made
+ * current or insufficient to render projectM visuals.
+ *
+ * Note: The OpenGL resolver is initialized on the first call to either projectm_create() or projectm_create_with_opengl_load_proc().
+ * All projectM instances share the same resolver, and subsequent calls ignore the provided load_proc.
+ *
+ * @return A projectM handle for the newly created instance that must be used in subsequent API calls.
+ *         NULL if the instance could not be created successfully.
+ * @since 4.2.0
+ */
+PROJECTM_EXPORT projectm_handle projectm_create_with_opengl_load_proc(void* (*load_proc)(const char*, void*), void* user_data);
+
 /**
  * @brief Destroys the given instance and frees the resources.
  *

src/libprojectM/CMakeLists.txt

@@ -6,6 +6,10 @@ add_compile_definitions(
         $<IF:$<PLATFORM_ID:Windows>,STBI_NO_DDS,>
         )
 
+include_directories(
+        "${PROJECTM_SOURCE_DIR}/vendor/glad/include"
+        )
+
 add_subdirectory(Audio)
 add_subdirectory(MilkdropPreset)
 add_subdirectory(Renderer)
@@ -70,6 +74,7 @@ add_library(projectM
         $<TARGET_OBJECTS:stb_image>
         $<TARGET_OBJECTS:projectM_main>
         $<TARGET_OBJECTS:projectM::Eval>
+        $<TARGET_OBJECTS:glad_obj>
         )
 
 target_include_directories(projectM
@@ -84,6 +89,7 @@ if(ENABLE_CXX_INTERFACE)
             )
 endif()
 
+# PROJECTM_OPENGL_LIBRARIES is present for WIN32/Desktop GL only
 target_link_libraries(projectM
         PUBLIC
         ${PROJECTM_OPENGL_LIBRARIES}
@@ -220,14 +226,6 @@ if(ENABLE_INSTALL)
             COMPONENT Devel
             )
 
-    if(NOT ENABLE_EMSCRIPTEN AND ENABLE_GLES)
-        install(FILES
-                "${PROJECTM_SOURCE_DIR}/cmake/gles/FindOpenGL.cmake"
-                DESTINATION "${PROJECTM_LIB_DIR}/cmake/projectM4"
-                COMPONENT Devel
-                )
-    endif()
-
     install(EXPORT libprojectMTargets
             FILE projectM4Targets.cmake
             NAMESPACE libprojectM::

src/libprojectM/MilkdropPreset/CMakeLists.txt

@@ -140,7 +140,6 @@ target_link_libraries(MilkdropPreset
         PUBLIC
         hlslparser
         GLM::GLM
-        ${PROJECTM_OPENGL_LIBRARIES}
         )
 
 if(BUILD_SHARED_LIBS)

src/libprojectM/ProjectMCWrapper.cpp

@@ -5,6 +5,7 @@
 #include <Logging.hpp>
 
 #include <Audio/AudioConstants.hpp>
+#include <Renderer/PlatformGLResolver.hpp>
 
 #include <projectM-4/parameters.h>
 #include <projectM-4/render_opengl.h>
@@ -67,10 +68,27 @@ void projectm_free_string(const char* str)
 }
 
 projectm_handle projectm_create()
+{
+    return projectm_create_with_opengl_load_proc(nullptr, nullptr);
+}
+
+projectm_handle projectm_create_with_opengl_load_proc(void* (*load_proc)(const char*, void*), void* user_data)
 {
     try
     {
-        auto projectMInstance = new libprojectM::projectMWrapper();
+        // obtain shared resolver instance
+        auto& glResolver = libprojectM::Renderer::Platform::GLResolver::Instance();
+
+        // init resolver to discover gl function pointers and init GLAD
+        // Initialize() is guarded internally, may be called multiple times
+        auto success = glResolver.Initialize(load_proc, user_data);
+        if (!success)
+        {
+            return nullptr;
+        }
+
+        // create projectM
+        auto* projectMInstance = new libprojectM::projectMWrapper();
         return reinterpret_cast<projectm_handle>(projectMInstance);
     }
     catch (...)

src/libprojectM/Renderer/CMakeLists.txt

@@ -32,6 +32,12 @@ add_library(Renderer OBJECT
         MilkdropNoise.cpp
         MilkdropNoise.hpp
         OpenGL.h
+        PlatformGLContextCheck.cpp
+        PlatformGLContextCheck.hpp
+        PlatformGLResolver.cpp
+        PlatformGLResolver.hpp
+        PlatformLoader.cpp
+        PlatformLoader.hpp
         Point.hpp
         PresetTransition.cpp
         PresetTransition.hpp
@@ -59,7 +65,7 @@ add_library(Renderer OBJECT
         VertexBufferUsage.hpp
         VertexIndexArray.cpp
         VertexIndexArray.hpp
-        )
+)
 
 target_include_directories(Renderer
         PRIVATE
@@ -76,6 +82,7 @@ target_link_libraries(Renderer
         GLM::GLM
         hlslparser
         stb_image
+        glad
         ${PROJECTM_OPENGL_LIBRARIES}
         )
 

src/libprojectM/Renderer/OpenGL.h

@@ -4,31 +4,8 @@
  */
 #pragma once
 
-#ifdef __APPLE__ /* macOS */
-#include <OpenGL/gl3.h>
-#include <OpenGL/gl3ext.h>
-#elif defined(EYETUNE_WINRT) /* Universal Windows Platform */
-#define GL_GLEXT_PROTOTYPES
-#define GLM_FORCE_CXX03
-#include <EGL/egl.h>
-#include <EGL/eglext.h>
-#include <GLES2/gl2.h>
-#include <GLES2/gl2ext.h>
-#include <GLES3/gl3.h>
-#include <GLES3/gl31.h>
-#elif defined(_WIN32) /* Windows Desktop */
-#define GLM_FORCE_CXX03
-#include <GL/glew.h>
-#include <GL/wglew.h>
-#include <windows.h>
-#else /* Linux, BSD, Android, emscripten etc. */
 #ifdef USE_GLES
-#include <GLES3/gl3.h>
+   #include <glad/gles2.h>
 #else
-#if !defined(GL_GLEXT_PROTOTYPES)
-#define GL_GLEXT_PROTOTYPES
-#endif
-#include <GL/gl.h>
-#include <GL/glext.h>
-#endif
+   #include <glad/gl.h>
 #endif