sdl12-compat: minor clean-up and sync with latest SDL2

From cc0201143a33a399a8d4f01571cfe5fceec09df5 Mon Sep 17 00:00:00 2001
From: Ozkan Sezer <[EMAIL REDACTED]>
Date: Thu, 14 Aug 2025 06:28:20 +0300
Subject: [PATCH] minor clean-up and sync with latest SDL2

---
 src/SDL12_compat.c          | 31 +++++++++++++++----------------
 src/SDL20_include_wrapper.h | 32 ++++++++++++++++++--------------
 2 files changed, 33 insertions(+), 30 deletions(-)

diff --git a/src/SDL12_compat.c b/src/SDL12_compat.c
index b3c2ec565..9b968d130 100644
--- a/src/SDL12_compat.c
+++ b/src/SDL12_compat.c
@@ -2413,7 +2413,7 @@ AddVidModeToList(VideoModeList *vmode, SDL12_Rect *mode, const Uint16 maxw, cons
     void *ptr = NULL;
     int i;
 
-    if ( (maxw && (mode->w > maxw)) || (maxh && (mode->h > maxh)) ) {
+    if ((maxw && (mode->w > maxw)) || (maxh && (mode->h > maxh))) {
         return 0;   /* clamp this one out as too big. */
     }
 
@@ -5970,7 +5970,7 @@ LoadOpenGLFunctions(void)
 }
 
 static void
-ResolveFauxBackbufferMSAA()
+ResolveFauxBackbufferMSAA(void)
 {
     const GLboolean has_scissor = OpenGLFuncs.glIsEnabled(GL_SCISSOR_TEST);
 
@@ -6234,15 +6234,7 @@ UnlockVideoRenderer(void)
     SDL20_UnlockMutex(VideoRendererLock);
 }
 
-static void HandleInputGrab(SDL12_GrabMode mode);
-
-
-/* SDL_SetRefreshRate was never in an real SDL-1.2 release, but apparently StepMania was maintaining a fork with this API for literally years. */
-DECLSPEC12 void SDLCALL
-SDL_SetRefreshRate(int rate)
-{
-    DesiredRefreshRate = (rate >= 0) ? rate : 0;  /* takes effect on next SDL_SetVideoMode call. */
-}
+static void HandleInputGrab(SDL12_GrabMode);
 
 static SDL12_Surface *
 SetVideoModeImpl(int width, int height, int bpp, Uint32 flags12)
@@ -6287,7 +6279,7 @@ SetVideoModeImpl(int width, int height, int bpp, Uint32 flags12)
         use_highdpi = (flags12 & SDL12_FULLSCREEN) ? use_gl_scaling : SDL_FALSE;
 
         gl_max_fps = SDL12Compat_GetHintInt("SDL12COMPAT_MAX_FPS", 0);
-        if(gl_max_fps != 0) {
+        if (gl_max_fps != 0) {
             OpenGLBuffersSwapTickInterval = 1000.f / gl_max_fps;
             OpenGLBuffersLastSwapTicks = SDL20_GetTicks();
         }
@@ -6719,6 +6711,13 @@ SDL_SetVideoMode(int width, int height, int bpp, Uint32 flags12)
     return retval;
 }
 
+/* SDL_SetRefreshRate was never in an real SDL-1.2 release, but apparently StepMania was maintaining a fork with this API for literally years. */
+DECLSPEC12 void SDLCALL
+SDL_SetRefreshRate(int rate)
+{
+    DesiredRefreshRate = (rate >= 0) ? rate : 0;  /* takes effect on next SDL_SetVideoMode call. */
+}
+
 DECLSPEC12 SDL12_Surface * SDLCALL
 SDL_GetVideoSurface(void)
 {
@@ -7132,7 +7131,7 @@ UpdateRect12to20(SDL12_Surface *surface12, const SDL12_Rect *rect12, SDL_Rect *r
 
 /* For manual throttling of screen updates. */
 static int
-GetDesiredMillisecondsPerFrame()
+GetDesiredMillisecondsPerFrame(void)
 {
     SDL_DisplayMode mode;
     if (VideoSurface12->flags & SDL12_FULLSCREEN) {
@@ -8213,9 +8212,9 @@ DECLSPEC12 void SDLCALL
 SDL_GL_SwapBuffers(void)
 {
     if (VideoWindow20) {
-        if(OpenGLBuffersSwapTickInterval != 0.f) {
+        if (OpenGLBuffersSwapTickInterval != 0.f) {
             const Uint32 tickDelta = SDL20_GetTicks() - OpenGLBuffersLastSwapTicks;
-            if(tickDelta < OpenGLBuffersSwapTickInterval) {
+            if (tickDelta < OpenGLBuffersSwapTickInterval) {
                 SDL20_Delay((int)(OpenGLBuffersSwapTickInterval + 0.5f) - tickDelta);
             }
         }
@@ -8269,7 +8268,7 @@ SDL_GL_SwapBuffers(void)
             SDL20_GL_SwapWindow(VideoWindow20);
         }
 
-        if(OpenGLBuffersSwapTickInterval != 0.f) {
+        if (OpenGLBuffersSwapTickInterval != 0.f) {
             OpenGLBuffersLastSwapTicks = SDL20_GetTicks();
         }
     }
diff --git a/src/SDL20_include_wrapper.h b/src/SDL20_include_wrapper.h
index 28eca11f1..50aa440a8 100644
--- a/src/SDL20_include_wrapper.h
+++ b/src/SDL20_include_wrapper.h
@@ -898,6 +898,7 @@
 #define SDL_HasWindowSurface IGNORE_THIS_VERSION_OF_SDL_HasWindowSurface
 #define SDL_DestroyWindowSurface IGNORE_THIS_VERSION_OF_SDL_DestroyWindowSurface
 #define SDL_GDKGetDefaultUser IGNORE_THIS_VERSION_OF_SDL_GDKGetDefaultUser
+#define SDL_GameControllerGetSteamHandle IGNORE_THIS_VERSION_OF_SDL_GameControllerGetSteamHandle
 
 
 #if defined(_WIN32) || defined(__OS2__)
@@ -948,18 +949,6 @@ typedef void (__cdecl *pfnSDL_CurrentEndThread) (unsigned);
 #endif /* _WIN32 */
 
 
-#ifdef SDL_BlitSurface
-#undef SDL_BlitSurface
-#endif
-
-#ifdef SDL_mutexP
-#undef SDL_mutexP
-#endif
-
-#ifdef SDL_mutexV
-#undef SDL_mutexV
-#endif
-
 #ifdef SDL_SetError
 #undef SDL_SetError
 #endif
@@ -4440,8 +4429,23 @@ typedef void (__cdecl *pfnSDL_CurrentEndThread) (unsigned);
 #undef SDL_GDKGetDefaultUser
 #endif
 
-/* undefine these macros too: */
-/* redefine using SDL2_xxx, if needed. */
+#ifdef SDL_GameControllerGetSteamHandle
+#undef SDL_GameControllerGetSteamHandle
+#endif
+
+/* undefine these macros, too: redefine as SDL2_xxx, if needed.
+ */
+#ifdef SDL_BlitSurface
+#undef SDL_BlitSurface
+#endif
+
+#ifdef SDL_mutexP
+#undef SDL_mutexP
+#endif
+
+#ifdef SDL_mutexV
+#undef SDL_mutexV
+#endif
 
 #ifdef SDL_enabled_assert
 #undef SDL_enabled_assert