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-hexdecoct.c | |
| parent | 6978efcffb1aaab67444afb9f0369582ec88996a (diff) | |
| download | systemd-dcd6361ea1dfaa6810b480e0bc53674e5b2c822d.tar.gz | |
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-hexdecoct.c')
| -rw-r--r-- | src/test/test-hexdecoct.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/test/test-hexdecoct.c b/src/test/test-hexdecoct.c index fb98826c27..41ca9805fe 100644 --- a/src/test/test-hexdecoct.c +++ b/src/test/test-hexdecoct.c @@ -85,10 +85,10 @@ static void test_unhexmem_one(const char *s, size_t l, int retval) { if (retval == 0) { char *answer; - if (l == (size_t) - 1) + if (l == (size_t) -1) l = strlen(s); - assert_se((hex = hexmem(mem, len))); + assert_se(hex = hexmem(mem, len)); answer = strndupa(s, l); assert_se(streq(delete_chars(answer, WHITESPACE), hex)); } |