From 4acdd2bef37ebff235a20a5d7b2b6715e139bf00 Mon Sep 17 00:00:00 2001
From: "Ryan C. Gordon" <[EMAIL REDACTED]>
Date: Mon, 24 Jan 2022 09:55:10 -0500
Subject: [PATCH] events: Don't send an SDL_QUIT for SDL_WINDOWEVENT_CLOSE.
Otherwise, since 1.2 only ever has one windows, we just end up sending two
SDL12_QUIT events, which confuses TuxPaint.
It's better to look for the SDL_QUIT event anyhow, since SIGINT on Unix,
Command-Q on macOS, etc, can trigger it, too.
Fixes #154.
---
src/SDL12_compat.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/SDL12_compat.c b/src/SDL12_compat.c
index ef62faa..57c0ed0 100644
--- a/src/SDL12_compat.c
+++ b/src/SDL12_compat.c
@@ -3665,9 +3665,9 @@ EventFilter20to12(void *data, SDL_Event *event20)
case SDL_WINDOWEVENT:
switch (event20->window.event) {
- case SDL_WINDOWEVENT_CLOSE:
- event12.type = SDL12_QUIT;
- break;
+ /* don't send an SDL12_QUIT event for SDL_WINDOWEVENT_CLOSE;
+ we only ever have a single window, so an SDL_QUIT will be
+ coming from SDL2 next anyhow, so just send that on. */
case SDL_WINDOWEVENT_SHOWN:
case SDL_WINDOWEVENT_EXPOSED: