[libvirt] [PATCHv2 11/13] list: provide RPC call for snapshots

Eric Blake eblake at redhat.com
Tue Jun 19 21:13:36 UTC 2012


On 06/18/2012 09:00 AM, Eric Blake wrote:
> On 06/18/2012 08:48 AM, Peter Krempa wrote:
>> On 06/15/12 06:18, Eric Blake wrote:
>>> The generator doesn't handle lists of virDomainSnapshotPtr, so
>>> this commit requires a bit more work than some RPC additions.
>>>
> 
>>> +struct remote_domain_snapshot_list_all_children_ret {
>>> +    remote_nonnull_domain_snapshot snapshots<>;
>>
>> Hopefully we'll go with unbounded arrays in the future, but as with my
>> listing function: Shouldn't we use the limit that was imposed on name
>> list length also on this list? (REMOTE_DOMAIN_SNAPSHOT_LIST_NAMES_MAX).
> 
> Here's a case where I was copying from your series :)
> 
> Whichever way you go in your commit, I will copy.

Looks like you used unbounded <>, so I copied.

>>
>> ACK with the limit added. I'm a fan of unbounded arrays, but I think we
>> should be consistent.

I will push this one as-is.

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



-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120619/3cc8f23b/attachment-0001.sig>


More information about the libvir-list mailing list