[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] add input-send-event command
From: |
Marcelo Tosatti |
Subject: |
Re: [Qemu-devel] [PATCH] add input-send-event command |
Date: |
Tue, 17 Jun 2014 10:41:23 -0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, Jun 17, 2014 at 05:54:35AM -0600, Eric Blake wrote:
> On 06/17/2014 03:02 AM, Gerd Hoffmann wrote:
> > On Mo, 2014-06-16 at 20:15 -0300, Marcelo Tosatti wrote:
> >> Which allows specification of absolute/relative,
> >> up/down and console parameters.
> >
> > Looks good overall, only minor nits. Will wait for acks from qmp guys
> > before picking into input branch.
> >
> >> Suggested by Gerd Hoffman.
> >
> > Huh? Signed-off missing ...
> >
> >> +Press left mouse button.
> >> +
> >> +-> { "execute": "input-send-event",
> >> + "arguments": { "console": 0,
> >> + "events": [ { "type": "btn",
> >> + "data" : { "down": true, "button": "Left" } },
> >> + { "type": "btn",
> >> + "data" : { "down": false, "button": "Left" } } ] } }
> >> +<- { "return": {} }
> >
> > Bad example. Down + up should go as separate commands (so you have a
> > sync inbetween), otherwise it might happen that the guest doesn't see
> > the button event at all.
>
> Would it be worth adding a command to the union of possible events that
> adds a pause? As in:
>
> -> { "execute": "input-send-event",
> "arguments": { "console": 0,
> "events": [
> { "type": "btn",
> "data" : { "down": true, "button": "Left" } },
> { "type": "pause", "data": { "duration": 10 } },
> { "type": "btn",
> "data" : { "down": false, "button": "Left" } } ] } }
>
> that sticks a 10 ms delay between the other two events?
Would rather have the user wait 10 ms in between QMP commands.