_dbus_test_check: Wrap body of macro in do/while

This avoids unexpected precedence when used as the body of an "if" or
"else" without being wrapped in {}.

Signed-off-by: Simon McVittie <smcv@collabora.com>

diff --git a/dbus/dbus-test-tap.h b/dbus/dbus-test-tap.h
index 423cf48ea8bd9041799c27369ed4a80e0818de30..19f041b6c969dadd5303576b1d660dd01f67421f 100644 (file)
--- a/dbus/dbus-test-tap.h
+++ b/dbus/dbus-test-tap.h
@@ -59,7 +59,10 @@ void _dbus_test_check_memleaks (const char *test_name);
 DBUS_PRIVATE_EXPORT
 int _dbus_test_done_testing (void);
 
-#define _dbus_test_check(a) if (!(a)) _dbus_test_not_ok ("%s:%d - '%s' failed\n", __FILE__, __LINE__, #a)
+#define _dbus_test_check(a) do { \
+    if (!(a)) \
+      _dbus_test_not_ok ("%s:%d - '%s' failed\n", __FILE__, __LINE__, #a); \
+  } while (0)
 
 #endif