Re: [libvirt PATCH 03/11] virxml: Simplify VIR_XPATH_NODE_AUTORESTORE

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

 



On a Wednesday in 2021, Tim Wiederhake wrote:
Signed-off-by: Tim Wiederhake <twiederh@xxxxxxxxxx>
---
src/util/virxml.h | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/util/virxml.h b/src/util/virxml.h
index 06fb7aebd8..b4716630b9 100644
--- a/src/util/virxml.h
+++ b/src/util/virxml.h
@@ -358,10 +358,8 @@ G_DEFINE_AUTO_CLEANUP_CLEAR_FUNC(virXPathContextNodeSave, virXPathContextNodeRes
 * node pointer is reset to the original value when this macro was used.
 */
#define VIR_XPATH_NODE_AUTORESTORE(_ctxt) \
-    VIR_WARNINGS_NO_UNUSED_VARIABLE \
-    g_auto(virXPathContextNodeSave) _ctxt ## CtxtSave = { .ctxt = _ctxt,\
-                                                          .node = _ctxt->node}; \
-    VIR_WARNINGS_RESET
+    vir_g_auto(virXPathContextNodeSave) _ctxt ## CtxtSave = { .ctxt = _ctxt, \
+                                                              .node = _ctxt->node}

Instead of creating the vir_g_auto macro, you can simply put the
G_GNUC_UNUSED in the declaration here:
g_auto(virXPathContextNodeSave) _ctxt ## CtxtSave G_GNUC_UNUSED = ...

(I don't remember why I did not figure that out in the first place.)

Jano

Attachment: signature.asc
Description: PGP signature


[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]

  Powered by Linux