Hi,
On 05-06-15 08:41, Peter Hutterer wrote:
Previous expansion had side-effects when litest_log was called in an if
condition without {}
Signed-off-by: Peter Hutterer <[email protected]>
The entire set looks good to me:
Reviewed-by: Hans de Goede <[email protected]>
Regards,
Hans
---
test/litest.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/test/litest.c b/test/litest.c
index c9c17d0..0e2cb96 100644
--- a/test/litest.c
+++ b/test/litest.c
@@ -61,8 +61,8 @@ const char *filter_group = NULL;
#define litest_log(...) fprintf(stderr, __VA_ARGS__)
#define litest_vlog(format_, args_) vfprintf(stderr, format_, args_)
#else
-#define litest_log(...) /* __VA_ARGS__ */
-#define litest_vlog(...) /* __VA_ARGS__ */
+#define litest_log(...) { /* __VA_ARGS__ */ }
+#define litest_vlog(...) { /* __VA_ARGS__ */ }
#endif
#ifdef HAVE_LIBUNWIND
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel
