mailinfo: work around -Wstring-plus-int warning
authorEric Sunshine <sunshine@sunshineco.com>
Sun, 21 Sep 2014 09:13:58 +0000 (05:13 -0400)
committerJunio C Hamano <gitster@pobox.com>
Mon, 22 Sep 2014 20:46:43 +0000 (13:46 -0700)
The just-released Apple Xcode 6.0.1 has -Wstring-plus-int enabled by
default which complains about pointer arithmetic applied to a string
literal:

builtin/mailinfo.c:303:24: warning:
adding 'long' to a string does not append to the string
return !memcmp(SAMPLE + (cp - line), cp, strlen(SAMPLE) ...
~~~~~~~^~~~~~~~~~~~~

Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/mailinfo.c
index 2632fb037a6a285b17ca89911dc3da17cab231dd..6a14d2985d12394d343cb91290111d6a8c0a5d06 100644 (file)
@@ -288,9 +288,10 @@ static inline int cmp_header(const struct strbuf *line, const char *hdr)
                        line->buf[len] == ':' && isspace(line->buf[len + 1]);
 }
 
-#define SAMPLE "From e6807f3efca28b30decfecb1732a56c7db1137ee Mon Sep 17 00:00:00 2001\n"
 static int is_format_patch_separator(const char *line, int len)
 {
+       static const char SAMPLE[] =
+               "From e6807f3efca28b30decfecb1732a56c7db1137ee Mon Sep 17 00:00:00 2001\n";
        const char *cp;
 
        if (len != strlen(SAMPLE))