[libvirt] [PATCH 1/5] virsh: Allow using VIR_AUTOPTR for releasing virDomainPtr in virsh

Peter Krempa pkrempa at redhat.com
Mon Sep 16 12:45:32 UTC 2019


I opted to alias the 'virDomainType' to 'virshDomain' so that it's
obvious in all cases that this is a virsh-only construct. This is also
somewhat consistent with virsh's use of 'virshDomainFree' wrapper for
the freeing function which actually accepts NULL.

Signed-off-by: Peter Krempa <pkrempa at redhat.com>
---
 tools/virsh-util.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tools/virsh-util.h b/tools/virsh-util.h
index 9005aa9d36..7fdd39dd12 100644
--- a/tools/virsh-util.h
+++ b/tools/virsh-util.h
@@ -39,8 +39,11 @@ virshCommandOptDomain(vshControl *ctl,
                       const vshCmd *cmd,
                       const char **name);

+typedef virDomain virshDomain;
+
 void
 virshDomainFree(virDomainPtr dom);
+VIR_DEFINE_AUTOPTR_FUNC(virshDomain, virshDomainFree);

 void
 virshDomainCheckpointFree(virDomainCheckpointPtr chk);
-- 
2.21.0




More information about the libvir-list mailing list