On 01/27/2015 12:46 PM, Max Reitz wrote:
> Signed-off-by: Max Reitz <mre...@redhat.com>
> ---
>  blockdev.c           | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
>  qapi/block-core.json | 21 +++++++++++++++++++++
>  qmp-commands.hx      | 37 +++++++++++++++++++++++++++++++++++++
>  3 files changed, 106 insertions(+)
> 

> +Example (1):
> +

With only one example, the (1) doesn't add much.  Minor whether you
leave it in or take it out.

> +-> { "execute": "blockdev-open-tray",
> +     "arguments": { "device": "ide1-cd0" } }
> +
> +<- { "timestamp": { "seconds": 1418751016,
> +                    "microseconds": 716996 },
> +     "event": "DEVICE_TRAY_MOVED",
> +     "data": { "device": "ide1-cd0",
> +               "tray-open": true } }
> +
> +<- { "return": {} }

Nice inclusion of the event alongside the return code!

Reviewed-by: Eric Blake <ebl...@redhat.com>

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to