diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2016-10-23 11:46:51 -0400 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2016-10-23 11:46:51 -0400 |
commit | f60dad309a815ef55d7688343c62fbbf9d8e1040 (patch) | |
tree | 53ab14b494fd2ffe556dbc3582c3c65f2e1b5f5d | |
parent | 605405c6cc934466951b0c6bad5a9553620bcb08 (diff) |
test-copy: wrap assignments in function call in parentheses
-rw-r--r-- | src/test/test-copy.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/test/test-copy.c b/src/test/test-copy.c index 91e4e5b409..91d2a0bcd4 100644 --- a/src/test/test-copy.c +++ b/src/test/test-copy.c @@ -106,7 +106,7 @@ static void test_copy_tree(void) { STRV_FOREACH(p, files) { _cleanup_free_ char *f; - assert_se(f = strappend(original_dir, *p)); + assert_se((f = strappend(original_dir, *p))); assert_se(mkdir_parents(f, 0755) >= 0); assert_se(write_string_file(f, "file", WRITE_STRING_FILE_CREATE) == 0); @@ -115,8 +115,8 @@ static void test_copy_tree(void) { STRV_FOREACH_PAIR(link, p, links) { _cleanup_free_ char *f, *l; - assert_se(f = strappend(original_dir, *p)); - assert_se(l = strappend(original_dir, *link)); + assert_se((f = strappend(original_dir, *p))); + assert_se((l = strappend(original_dir, *link))); assert_se(mkdir_parents(l, 0755) >= 0); assert_se(symlink(f, l) == 0); @@ -131,7 +131,7 @@ static void test_copy_tree(void) { _cleanup_free_ char *buf = NULL, *f; size_t sz = 0; - assert_se(f = strappend(copy_dir, *p)); + assert_se((f = strappend(copy_dir, *p))); assert_se(access(f, F_OK) == 0); assert_se(read_full_file(f, &buf, &sz) == 0); @@ -141,8 +141,8 @@ static void test_copy_tree(void) { STRV_FOREACH_PAIR(link, p, links) { _cleanup_free_ char *target = NULL, *f, *l; - assert_se(f = strjoin(original_dir, *p)); - assert_se(l = strjoin(copy_dir, *link)); + assert_se((f = strjoin(original_dir, *p))); + assert_se((l = strjoin(copy_dir, *link))); assert_se(readlink_and_canonicalize(l, &target) == 0); assert_se(path_equal(f, target)); |