From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34241) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XzMuJ-0004sm-Ld for qemu-devel@nongnu.org; Fri, 12 Dec 2014 04:53:14 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XzMuC-0003cj-33 for qemu-devel@nongnu.org; Fri, 12 Dec 2014 04:53:07 -0500 Received: from mx1.redhat.com ([209.132.183.28]:43808) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XzMuB-0003cI-Qg for qemu-devel@nongnu.org; Fri, 12 Dec 2014 04:53:00 -0500 From: Max Reitz Date: Fri, 12 Dec 2014 10:52:52 +0100 Message-Id: <1418377972-25278-3-git-send-email-mreitz@redhat.com> In-Reply-To: <1418377972-25278-1-git-send-email-mreitz@redhat.com> References: <1418377972-25278-1-git-send-email-mreitz@redhat.com> Subject: [Qemu-devel] [PATCH 2/2] sdl2: Work around SDL2 SDL_ShowWindow() bug List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Gerd Hoffmann , Anthony Liguori , Max Reitz 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 --- ui/sdl2.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/ui/sdl2.c b/ui/sdl2.c index fdc762f..7297c02 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.9.3