From 10b524c7ccae6832ccd5106d409beb412b1e83f3 Mon Sep 17 00:00:00 2001 From: "Ryan C. Gordon" Date: Fri, 13 Feb 2026 14:31:45 -0500 Subject: [PATCH] hints: Rename SDL_HINT_OPENGL_FORCE_SRGB_CAPABLE. It's now SDL_HINT_OPENGL_FORCE_SRGB_FRAMEBUFFER, since it does more than mess with the srgb-capable context attribute now. Reference Issue #14898. --- include/SDL3/SDL_hints.h | 2 +- src/video/SDL_egl.c | 2 +- src/video/SDL_video.c | 2 +- src/video/uikit/SDL_uikitopengles.m | 2 +- src/video/windows/SDL_windowsopengl.c | 2 +- src/video/x11/SDL_x11opengl.c | 2 +- 6 files changed, 6 insertions(+), 6 deletions(-) diff --git a/include/SDL3/SDL_hints.h b/include/SDL3/SDL_hints.h index 4e07a173b4..5ab161f932 100644 --- a/include/SDL3/SDL_hints.h +++ b/include/SDL3/SDL_hints.h @@ -3078,7 +3078,7 @@ extern "C" { * * \since This hint is available since SDL 3.4.2. */ -#define SDL_HINT_OPENGL_FORCE_SRGB_CAPABLE "SDL_OPENGL_FORCE_SRGB_CAPABLE" +#define SDL_HINT_OPENGL_FORCE_SRGB_FRAMEBUFFER "SDL_OPENGL_FORCE_SRGB_FRAMEBUFFER" /** * Mechanism to specify openvr_api library location diff --git a/src/video/SDL_egl.c b/src/video/SDL_egl.c index fc5dfc95f3..6a3a4d85c4 100644 --- a/src/video/SDL_egl.c +++ b/src/video/SDL_egl.c @@ -1291,7 +1291,7 @@ EGLSurface SDL_EGL_CreateSurface(SDL_VideoDevice *_this, SDL_Window *window, Nat #ifdef EGL_KHR_gl_colorspace if (SDL_EGL_HasExtension(_this, SDL_EGL_DISPLAY_EXTENSION, "EGL_KHR_gl_colorspace")) { - const char *srgbhint = SDL_GetHint(SDL_HINT_OPENGL_FORCE_SRGB_CAPABLE); + const char *srgbhint = SDL_GetHint(SDL_HINT_OPENGL_FORCE_SRGB_FRAMEBUFFER); if (srgbhint && *srgbhint) { if (SDL_strcmp(srgbhint, "skip") == 0) { // don't set an attribute at all. diff --git a/src/video/SDL_video.c b/src/video/SDL_video.c index 172a6ff9a3..936aba705c 100644 --- a/src/video/SDL_video.c +++ b/src/video/SDL_video.c @@ -5396,7 +5396,7 @@ SDL_GLContext SDL_GL_CreateContext(SDL_Window *window) #if defined(SDL_VIDEO_OPENGL) || defined(SDL_VIDEO_OPENGL_ES) || defined(SDL_VIDEO_OPENGL_ES2) int srgb_requested = -1; - const char *srgbhint = SDL_GetHint(SDL_HINT_OPENGL_FORCE_SRGB_CAPABLE); + const char *srgbhint = SDL_GetHint(SDL_HINT_OPENGL_FORCE_SRGB_FRAMEBUFFER); if (srgbhint && *srgbhint) { srgb_requested = SDL_GetStringBoolean(srgbhint, false) ? 1 : 0; } diff --git a/src/video/uikit/SDL_uikitopengles.m b/src/video/uikit/SDL_uikitopengles.m index 20156a7671..db67a8e91c 100644 --- a/src/video/uikit/SDL_uikitopengles.m +++ b/src/video/uikit/SDL_uikitopengles.m @@ -155,7 +155,7 @@ SDL_GLContext UIKit_GL_CreateContext(SDL_VideoDevice *_this, SDL_Window *window) } int srgb = _this->gl_config.framebuffer_srgb_capable; - const char *srgbhint = SDL_GetHint(SDL_HINT_OPENGL_FORCE_SRGB_CAPABLE); + const char *srgbhint = SDL_GetHint(SDL_HINT_OPENGL_FORCE_SRGB_FRAMEBUFFER); if (srgbhint && *srgbhint) { srgb = SDL_GetStringBoolean(srgbhint, false) ? 1 : 0; // there is no "skip" here, since initWithFrame expects it, so we'll treat it as false. } diff --git a/src/video/windows/SDL_windowsopengl.c b/src/video/windows/SDL_windowsopengl.c index 7e9045b94d..f7b22ed0f0 100644 --- a/src/video/windows/SDL_windowsopengl.c +++ b/src/video/windows/SDL_windowsopengl.c @@ -661,7 +661,7 @@ static bool WIN_GL_SetupWindowInternal(SDL_VideoDevice *_this, SDL_Window *windo } if (_this->gl_data->HAS_WGL_ARB_framebuffer_sRGB) { - const char *srgbhint = SDL_GetHint(SDL_HINT_OPENGL_FORCE_SRGB_CAPABLE); + const char *srgbhint = SDL_GetHint(SDL_HINT_OPENGL_FORCE_SRGB_FRAMEBUFFER); if (srgbhint && *srgbhint) { if (SDL_strcmp(srgbhint, "skip") == 0) { // don't set an attribute at all. diff --git a/src/video/x11/SDL_x11opengl.c b/src/video/x11/SDL_x11opengl.c index 2654b99301..07e895d46c 100644 --- a/src/video/x11/SDL_x11opengl.c +++ b/src/video/x11/SDL_x11opengl.c @@ -585,7 +585,7 @@ static int X11_GL_GetAttributes(SDL_VideoDevice *_this, Display *display, int sc } if (_this->gl_data->HAS_GLX_ARB_framebuffer_sRGB) { - const char *srgbhint = SDL_GetHint(SDL_HINT_OPENGL_FORCE_SRGB_CAPABLE); + const char *srgbhint = SDL_GetHint(SDL_HINT_OPENGL_FORCE_SRGB_FRAMEBUFFER); if (srgbhint && *srgbhint) { if (SDL_strcmp(srgbhint, "skip") == 0) { // don't set an attribute at all.