From 7d4b2107a1fc1bb0a386329c122b9a915177fe1d Mon Sep 17 00:00:00 2001
From: Carlo Bramini <[EMAIL REDACTED]>
Date: Thu, 26 Aug 2021 12:29:09 +0200
Subject: [PATCH] [WINCE] Fix build with Embedded Visual Studio.
Embedded Visual Studio is C90.
This means that you cannot declare variables or externs where you want inside the code, but it must be on top of a "{...}" block.
So, the best way for fixing this error is to add braces to the "if" block inside SDL_VIDEO_DRIVER_GAPI and declare the extern GapiTransform() there.
Next, I received an error because WM_WINDOWPOSCHANGING is undeclared.
I suggest to add an "#ifdef WM_WINDOWPOSCHANGING ... #endif", in a similar manner it has been done for other events.
---
src/video/wincommon/SDL_sysevents.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/video/wincommon/SDL_sysevents.c b/src/video/wincommon/SDL_sysevents.c
index 207587e52..875e39128 100644
--- a/src/video/wincommon/SDL_sysevents.c
+++ b/src/video/wincommon/SDL_sysevents.c
@@ -446,9 +446,10 @@ LRESULT CALLBACK WinMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
x = (Sint16)LOWORD(lParam);
y = (Sint16)HIWORD(lParam);
#ifdef SDL_VIDEO_DRIVER_GAPI
- extern void GapiTransform(GapiInfo*, LONG*, LONG*);
- if (SDL_VideoSurface && this->hidden->gapiInfo)
+ if (SDL_VideoSurface && this->hidden->gapiInfo) {
+ extern void GapiTransform(GapiInfo*, LONG*, LONG*);
GapiTransform(this->hidden->gapiInfo, &x, &y);
+ }
#endif
}
posted = SDL_PrivateMouseButton(
@@ -561,6 +562,7 @@ LRESULT CALLBACK WinMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
return(0);
#endif /* WM_GETMINMAXINFO */
+#ifdef WM_WINDOWPOSCHANGING
case WM_WINDOWPOSCHANGING: {
WINDOWPOS *windowpos = (WINDOWPOS*)lParam;
@@ -576,6 +578,7 @@ LRESULT CALLBACK WinMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
}
}
return(0);
+#endif /* WM_WINDOWPOSCHANGING */
case WM_WINDOWPOSCHANGED: {
SDL_VideoDevice *this = current_video;