Re: [Qemu-devel] patch: add delay= suboption to -display curses

2014-05-26 Thread Gerd Hoffmann
On Sa, 2014-05-24 at 21:21 -0400, Dave Mielke wrote: > [quoted lines by Peter Maydell on 2014/05/25 at 01:04 +0100] > > >Why is this a problem only for the curses UI frontend, and not for > >any of the other UIs which might send key events? > > One reason is that most UIs send key events as they

Re: [Qemu-devel] patch: add delay= suboption to -display curses

2014-05-25 Thread Dave Mielke
[quoted lines by Peter Maydell on 2014/05/25 at 10:11 +0100] >Ah, I see. Still, I think it makes more sense for the queue and delay >to be in the common key handling code, not in the curses frontend >specifically. The code has been moved. I can see a couple of possibilities insofar as an option

Re: [Qemu-devel] patch: add delay= suboption to -display curses

2014-05-25 Thread Dave Mielke
[quoted lines by Peter Maydell on 2014/05/25 at 10:11 +0100] >Ah, I see. Still, I think it makes more sense for the queue and delay >to be in the common key handling code, not in the curses frontend >specifically. Yes, you're right. While the curses UI is especially vulnerable to the problem, ot

Re: [Qemu-devel] patch: add delay= suboption to -display curses

2014-05-25 Thread Peter Maydell
On 25 May 2014 02:21, Dave Mielke wrote: > [quoted lines by Peter Maydell on 2014/05/25 at 01:04 +0100] >>Why is this a problem only for the curses UI frontend, and not for >>any of the other UIs which might send key events? > > One reason is that most UIs send key events as they receive them from

Re: [Qemu-devel] patch: add delay= suboption to -display curses

2014-05-24 Thread Dave Mielke
[quoted lines by Peter Maydell on 2014/05/25 at 01:04 +0100] >Why is this a problem only for the curses UI frontend, and not for >any of the other UIs which might send key events? One reason is that most UIs send key events as they receive them from the keyboard, one at a time, whereas the curse

Re: [Qemu-devel] patch: add delay= suboption to -display curses

2014-05-24 Thread Peter Maydell
On 25 May 2014 00:29, Dave Mielke wrote: > The attached patch (qemu-curses-delay-1.patch) allows the user to specify that > he needs -display curses to insert a delay in between key events. The current > behaviour is that it inserts key events immediately, one right after another, > which has prov

[Qemu-devel] patch: add delay= suboption to -display curses

2014-05-24 Thread Dave Mielke
The attached patch (qemu-curses-delay-1.patch) allows the user to specify that he needs -display curses to insert a delay in between key events. The current behaviour is that it inserts key events immediately, one right after another, which has proven to be too fast for some applications. Please