Skip to content

Conversation

@adriengivry
Copy link
Member

@adriengivry adriengivry commented Apr 15, 2025

Description

Removed GLEW for GLAD (core profile 4.5, no extension).
Sources generated from: https://glad.dav1d.de

Review Guidance

Had to fix a bunch of things in OvRendering that was relying on non-standard EXT version of some OpenGL functions (i.e. glGenerateTextureMipmapEXT, and other ones).

GLEW vs GLAD

From our friend ChatGPT

GLEW

  • Mature and widely used.
  • Great for legacy OpenGL (pre-3.3).
  • Requires linking against a library (glew32.lib, etc.).
  • Loads all extensions, even unused ones.
  • Needs glewInit() after creating an OpenGL context.
  • Can have issues with core profile on macOS.
  • Less customizable.

GLAD

  • Modern, lightweight, and customizable.
  • You generate only what you need (version, profile, extensions).
  • No need to link any external lib—just include .c and .h.
  • Loads only what you specify, making it faster.
  • Works well with core profile and newer OpenGL versions.
  • Better suited for modern OpenGL (3.3+) and cross-platform development.

Bottom line:

  • Pick GLEW for old-school simplicity and legacy support.
  • Pick GLAD for modern, efficient OpenGL projects.

To-Do

Related Issues

Related to #459
Related to #436

@adriengivry adriengivry added Graphics Graphical feature Project Configuration Anything related to setuping our projects Cleanup Code cleanup labels Apr 15, 2025
@adriengivry adriengivry marked this pull request as ready for review April 30, 2025 13:38
@adriengivry adriengivry merged commit 48bffaf into main Apr 30, 2025
1 check passed
@adriengivry adriengivry deleted the feature/glad_sources branch April 30, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Cleanup Code cleanup Graphics Graphical feature Project Configuration Anything related to setuping our projects

Development

Successfully merging this pull request may close these issues.

3 participants