On 08/03/2016 04:10 AM, Peter Krempa wrote:
VIR_STEAL copies the second argument into the first and then sets it
to
NULL. This is useful for stealing pointers.
---
src/internal.h | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/src/internal.h b/src/internal.h
index 0dc34c7..c633ee6 100644
--- a/src/internal.h
+++ b/src/internal.h
@@ -307,6 +307,18 @@
} while (0)
/**
+ * VIR_STEAL:
+ *
+ * Steals pointer passed as second argument into the first argument. Second
+ * argument must not have side effects.
+ */
+# define VIR_STEAL(a, b) \
+ do { \
+ (a) = (b); \
+ (b) = NULL;\
+ } while (0)
+
+/**
* virCheckFlags:
* @supported: an OR'ed set of supported flags
* @retval: return value in case unsupported flags were passed
While one can look at the code and deduce what it does - the name is not
that descriptive. VIR_ASSIGN_PTR or VIR_ASSIGN_POINTER? Just a thought...
It's also not used until patch 9 - perhaps move it a bit closer to that.
John