Re: [PATCH] snapshot: add virDomainSnapshotRef API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/08/2012 10:38 AM, Daniel P. Berrange wrote:
> On Fri, Jun 08, 2012 at 10:33:42AM -0600, Eric Blake wrote:
>> virDomainSnapshotPtr has a refcount member, but no one was able
>> to use it.  Furthermore, all of our other vir*Ptr objects have
>> a *Ref method to match their *Free method.  Thankfully, this is
>> client-side only, so we can use this new function regardless of
>> how old the server side is!  (I have future patches to virsh
>> that want to use it.)
>>
>> * include/libvirt/libvirt.h.in (virDomainSnapshotRef): Declare.
>> * src/libvirt.c (virDomainSnapshotRef): Implement it.
>> * src/libvirt_public.syms (LIBVIRT_0.9.13): Export it.
>> ---
>>  include/libvirt/libvirt.h.in |    1 +
>>  src/libvirt.c                |   33 +++++++++++++++++++++++++++++++++
>>  src/libvirt_public.syms      |    5 +++++
>>  3 files changed, 39 insertions(+)
> 
> ACK

Thanks; pushed.

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

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]