diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2018-05-14 09:12:12 +0200 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2018-05-14 17:22:22 +0200 |
| commit | dcd6361ea1dfaa6810b480e0bc53674e5b2c822d (patch) | |
| tree | 8f05ef56358b25789dd2e97002732efa78de10cf /src/test/test-unit-file.c | |
| parent | core/mount-setup: remove part of check which is always true (diff) | |
| download | systemd-dcd6361ea1dfaa6810b480e0bc53674e5b2c822d.tar.xz systemd-dcd6361ea1dfaa6810b480e0bc53674e5b2c822d.zip | |
tree-wide: do not wrap assert_se in extra parentheses
We were inconsitently using them in some cases, but in majority not.
Using assignment in assert_se is very common, not an exception like in
'if', so let's drop the extra parens everywhere.
Diffstat (limited to 'src/test/test-unit-file.c')
| -rw-r--r-- | src/test/test-unit-file.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/test/test-unit-file.c b/src/test/test-unit-file.c index 3fd4ac9e5d..13db2f4225 100644 --- a/src/test/test-unit-file.c +++ b/src/test/test-unit-file.c @@ -636,8 +636,8 @@ static void test_install_printf(void) { assert_se(specifier_machine_id('m', NULL, NULL, &mid) >= 0 && mid); assert_se(specifier_boot_id('b', NULL, NULL, &bid) >= 0 && bid); - assert_se((host = gethostname_malloc())); - assert_se((user = uid_to_name(getuid()))); + assert_se(host = gethostname_malloc()); + assert_se(user = uid_to_name(getuid())); assert_se(asprintf(&uid, UID_FMT, getuid()) >= 0); #define expect(src, pattern, result) \ |