diff options
| author | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2011-08-05 13:40:44 +0100 |
|---|---|---|
| committer | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2011-09-19 13:50:38 +0100 |
| commit | 86b0ba0278f527143fe5785ca819e45b94937474 (patch) | |
| tree | c1f0bcc17e16505a405c706e7b1e4288941862bb /bus/activation.c | |
| parent | d1756a7e4cbb6ef83b70319ab72a49760f2c227e (diff) | |
| download | dbus-86b0ba0278f527143fe5785ca819e45b94937474.tar.gz | |
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.
(<https://bugs.freedesktop.org/show_bug.cgi?id=39858>)
Diffstat (limited to 'bus/activation.c')
| -rw-r--r-- | bus/activation.c | 8 |
1 files changed, 6 insertions, 2 deletions
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)) |