From 86b0ba0278f527143fe5785ca819e45b94937474 Mon Sep 17 00:00:00 2001 From: Simon McVittie Date: Fri, 5 Aug 2011 13:40:44 +0100 Subject: Use {}, not semicolon, when the statement of an "if" does nothing The uses in bus/activation.c are also probably wrong because they ignore the result of the test, but that's orthogonal. () --- bus/activation.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'bus/activation.c') diff --git a/bus/activation.c b/bus/activation.c index 6101173f..dea62433 100644 --- a/bus/activation.c +++ b/bus/activation.c @@ -2535,14 +2535,18 @@ bus_activation_service_reload_test (const DBusString *test_data_dir) _dbus_assert_not_reached ("could not initiate service reload test"); if (!do_service_reload_test (&directory, FALSE)) - ; /* Do nothing? */ + { + /* Do nothing? */ + } /* Do OOM tests */ if (!init_service_reload_test (&directory)) _dbus_assert_not_reached ("could not initiate service reload test"); if (!do_service_reload_test (&directory, TRUE)) - ; /* Do nothing? */ + { + /* Do nothing? */ + } /* Cleanup test directory */ if (!cleanup_service_reload_test (&directory)) -- cgit v1.2.1