do not check truth value of flex arrays
authorJeff King <redacted>
Wed, 28 Jan 2015 17:58:50 +0000 (12:58 -0500)
committerJunio C Hamano <redacted>
Wed, 28 Jan 2015 20:46:07 +0000 (12:46 -0800)
There is no point in checking "!ref->name" when ref is a
"struct ref". The name field is a flex-array, and there
always has a non-zero address. This is almost certainly not
hurting anything, but it does cause clang-3.6 to complain.

Signed-off-by: Jeff King <redacted>
Signed-off-by: Junio C Hamano <redacted>
remote-curl.c

index 0eb6fc48fbb8f9c599386381539bd2fa4d23e40e..38cab05ec52585c886e09f58acb2511c9209cbcb 100644 (file)
@@ -514,7 +514,7 @@ static int fetch_git(struct discovery *heads,
        argv[argc++] = url;
        for (i = 0; i < nr_heads; i++) {
                struct ref *ref = to_fetch[i];
-               if (!ref->name || !*ref->name)
+               if (!*ref->name)
                        die("cannot fetch by sha1 over smart http");
                argv[argc++] = ref->name;
        }
git clone https://git.99rst.org/PROJECT