[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 12/12] sdl2: Work around SDL2 SDL_ShowWindow() bug
From: |
Gerd Hoffmann |
Subject: |
[Qemu-devel] [PULL 12/12] sdl2: Work around SDL2 SDL_ShowWindow() bug |
Date: |
Fri, 19 Dec 2014 10:17:06 +0100 |
From: Max Reitz <address@hidden>
Apparently it is possible for X to send an event to a hidden SDL2
window, leading to SDL2 believing it is now shown. SDL2 will pass the
SDL_WINDOWEVENT_SHOWN message to the application without actually
showing the window; the problem is that the next SDL_ShowWindow() will
be a no-op because SDL2 assumes the window is already shown.
The correct way to react to SDL_WINDOWEVENT_SHOWN would be to clear
scon->hidden (analogous for SDL_WINDOWEVENT_HIDDEN). However, due to the
window not actually being shown, this will somehow not be correct after
all.
Therefore, just hide the window on SDL_WINDOWEVENT_SHOWN if it is
supposed to be hidden (and analogous for SDL_WINDOWEVENT_HIDDEN).
Signed-off-by: Max Reitz <address@hidden>
Signed-off-by: Gerd Hoffmann <address@hidden>
---
ui/sdl2.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/ui/sdl2.c b/ui/sdl2.c
index 04bbcc7..1ae2781 100644
--- a/ui/sdl2.c
+++ b/ui/sdl2.c
@@ -548,6 +548,16 @@ static void handle_windowevent(SDL_Event *ev)
qemu_system_shutdown_request();
}
break;
+ case SDL_WINDOWEVENT_SHOWN:
+ if (scon->hidden) {
+ SDL_HideWindow(scon->real_window);
+ }
+ break;
+ case SDL_WINDOWEVENT_HIDDEN:
+ if (!scon->hidden) {
+ SDL_ShowWindow(scon->real_window);
+ }
+ break;
}
}
--
1.8.3.1
- [Qemu-devel] [PULL 00/12] sdl2: fixes, cleanups and opengl preparation., Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 02/12] sdl2: rename sdl2_state to sdl2_console, move to header file, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 03/12] sdl2: move keyboard input code to new sdl2-input.c, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 10/12] sdl2: move sdl2_2d_refresh to sdl2-2d.c, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 06/12] sdl2: overhaul window size handling, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 12/12] sdl2: Work around SDL2 SDL_ShowWindow() bug,
Gerd Hoffmann <=
- [Qemu-devel] [PULL 11/12] sdl2: Use correct sdl2_console for window events, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 09/12] sdl2: factor out sdl2_poll_events, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 05/12] sdl2: move sdl_update to new sdl2-2d.c, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 08/12] sdl2: add+use sdl2_2d_redraw function., Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 04/12] sdl2: turn on keyboard grabs, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 01/12] sdl: move version logic from source code to makefile, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 07/12] sdl2: move sdl_switch to sdl2-2d.c, Gerd Hoffmann, 2014/12/19
- Re: [Qemu-devel] [PULL 00/12] sdl2: fixes, cleanups and opengl preparation., Peter Maydell, 2014/12/22