[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main w
From: |
Jindřich Makovička |
Subject: |
Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main window |
Date: |
Wed, 08 Nov 2017 19:15:19 +0000 |
Hi Gerd,
I will try to respin the patch set during the weekend.
Regards, Jindrich
On Wed, Nov 8, 2017 at 11:10 AM Gerd Hoffmann <address@hidden> wrote:
> On Sun, Nov 05, 2017 at 09:10:45AM -0500, Cole Robinson wrote:
> > On 11/02/2017 01:12 PM, Jindřich Makovička wrote:
> > > This fixes the following case:
> > >
> > > 1) Ctrl-Alt-2 for console is pressed
> > > 2) console pops up and gets focus
> > > 3) console receives the Ctrl-Alt-2 keypress event
> > > 4) console closes
> > > 5) focus returns to the main window
> > > 6) main window gets the keypress event
> > > 7) goto 2 until the key is released, with 50% chance that the console
> > > window is shown at the end
> >
> > Sounds similar to what I reported in my comment here a while back:
> > https://bugzilla.libsdl.org/show_bug.cgi?id=3287
>
> Hmm, so it seems to be a SDL bug. I'm running SDL 2.0.3 here, which
> explains why I can't reproduce it. According to the bug it seem to be
> *repeat* keydown events, so is simply ignoring repeat events enough to
> avoid this issue?
>
> Also: Jindřich, do you plan to send a new of the patch series with the
> issues discussed fixed?
>
> cheers,
> Gerd
>
> --
Jindřich Makovička
- Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main window, (continued)
- Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main window, Gerd Hoffmann, 2017/11/03
- Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main window, Peter Maydell, 2017/11/03
- Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main window, Daniel P. Berrange, 2017/11/03
- Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main window, Gerd Hoffmann, 2017/11/03
- Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main window, Peter Maydell, 2017/11/03
- Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main window, Gerd Hoffmann, 2017/11/03
- Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main window, Peter Maydell, 2017/11/03
- Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main window, Gerd Hoffmann, 2017/11/03
Re: [Qemu-devel] [PATCH 4/7] sdl2: Only accept the hotkeys on the main window, Cole Robinson, 2017/11/05