A selection of refactors and improvements from the blockdev-add saga

[libvirt] [PATCH 03/11] util: uri: Introduce VIR_AUTOPTR freeing function

Posted by Peter Krempa 6 years, 10 months ago

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
---
 src/util/viruri.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/util/viruri.h b/src/util/viruri.h
index b6e97dafe6..f5b472860e 100644
--- a/src/util/viruri.h
+++ b/src/util/viruri.h
@@ -25,6 +25,7 @@

 # include "internal.h"
 # include "virconf.h"
+# include "virautoclean.h"

 typedef struct _virURI virURI;
 typedef virURI *virURIPtr;
@@ -60,6 +61,7 @@ char *virURIFormat(virURIPtr uri)
 char *virURIFormatParams(virURIPtr uri);

 void virURIFree(virURIPtr uri);
+VIR_DEFINE_AUTOPTR_FUNC(virURI, virURIFree);
 int virURIResolveAlias(virConfPtr conf, const char *alias, char **uri);

 # define VIR_URI_SERVER(uri) ((uri) && (uri)->server ? (uri)->server : "localhost")
-- 
2.20.1

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

Re: [libvirt] [PATCH 03/11] util: uri: Introduce VIR_AUTOPTR freeing function

Posted by Ján Tomko 6 years, 10 months ago

On Fri, Apr 05, 2019 at 06:04:15PM +0200, Peter Krempa wrote:
>Signed-off-by: Peter Krempa <pkrempa@redhat.com>
>---
> src/util/viruri.h | 2 ++
> 1 file changed, 2 insertions(+)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

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