From: Simon Marchi <simon.marchi@efficios.com>
Date: Thu, 9 May 2024 16:03:31 +0000 (-0400)
Subject: common: remove `BT_MOVE_REF`
X-Git-Url: http://git.efficios.com/?a=commitdiff_plain;h=41ff105c20a68584799523f69dd947325c755f92;p=babeltrace.git

common: remove `BT_MOVE_REF`

Change-Id: I458cbcdf8829a99f3290e0d76db2e13c23b1776f
Signed-off-by: Simon Marchi <simon.marchi@efficios.com>
Reviewed-on: https://review.lttng.org/c/babeltrace/+/12537
Tested-by: jenkins <jenkins@lttng.org>
Reviewed-by: Philippe Proulx <eeppeliteloop@gmail.com>
---

diff --git a/src/common/macros.h b/src/common/macros.h
index 8120515c..0f2bc8fb 100644
--- a/src/common/macros.h
+++ b/src/common/macros.h
@@ -45,29 +45,6 @@ extern "C" {
 #define BT_IF_DEV_MODE(txt)
 #endif
 
-/*
- * Yield `ref`'s value while setting `ref` to NULL.
- *
- * This can be used to give a strong reference to a callee:
- *
- *   add_foo_to_list(list, BT_MOVE_REF(foo));
- *
- * or in a simple assignment:
- *
- *   my_struct->foo = BT_MOVE_REF(foo);
- *
- * When moving a reference in a function call, the reference is given to the
- * callee even if that function call fails, so make sure the called function
- * is written accordingly.
- */
-
-#define BT_MOVE_REF(ref) 		\
-	({				\
-		__typeof__(ref) _ref = ref;	\
-		ref = NULL;		\
-		_ref;			\
-	})
-
 /* Wrapper for g_array_index that adds bound checking.  */
 #define bt_g_array_index(a, t, i)		\
 	g_array_index((a), t, ({ BT_ASSERT_DBG((i) < (a)->len); (i); }))