[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/6] rename blockdev-group-snapshot-sync
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 3/6] rename blockdev-group-snapshot-sync |
Date: |
Wed, 29 Feb 2012 16:23:20 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.1) Gecko/20120216 Thunderbird/10.0.1 |
Il 29/02/2012 16:08, Luiz Capitulino ha scritto:
>> >
>> > but it can just be omitted, I wasn't sure of how QAPI handled optionals.
> Do you now?
Perhaps not. :)
> All optionals will be accompanied of a 'bool has_OPTIONAL_NAME', this bool
> will be true if the optional has been passed by the caller/client or false
> otherwise (in which case you shouldn't trust it).
My understanding was that in this case I can trust the value to be
all-zeros (zero, false, 0.0, NULL), at least in the context of QAPI.
The QmpInputVisitor uses g_malloc0.
Paolo
- [Qemu-devel] [PATCH 0/6] Mirrored writes using blockdev-transaction, Paolo Bonzini, 2012/02/29
- [Qemu-devel] [PATCH 1/6] fix format name for backing file, Paolo Bonzini, 2012/02/29
- [Qemu-devel] [PATCH 2/6] qapi: complete implementation of unions, Paolo Bonzini, 2012/02/29
- [Qemu-devel] [PATCH 5/6] Add blkmirror block driver, Paolo Bonzini, 2012/02/29
- [Qemu-devel] [PATCH 3/6] rename blockdev-group-snapshot-sync, Paolo Bonzini, 2012/02/29
- Re: [Qemu-devel] [PATCH 3/6] rename blockdev-group-snapshot-sync, Eric Blake, 2012/02/29
- [Qemu-devel] [PATCH 6/6] add mirroring to blockdev-transaction, Paolo Bonzini, 2012/02/29
- [Qemu-devel] [PATCH 4/6] add reuse field, Paolo Bonzini, 2012/02/29