diff options
| author |  René Scharfe <l.s.r@web.de> | 2014-06-06 19:24:48 +0200 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2014-06-09 14:38:12 -0700 |
| commit | 50e19a83585bd1aeae6aa01d31e5f783b7ef8b1b (patch) | |
| tree | 37c4a5fad81436a9afc07e87486a26ec8b0b9e9b /builtin | |
| parent | Sync with 1.9.4 (diff) | |
| download | tgif-50e19a83585bd1aeae6aa01d31e5f783b7ef8b1b.tar.xz | |
Use starts_with() for C strings instead of memcmp()
Convert three cases of checking for a constant prefix using memcmp() to
starts_with(). This way there is no need for magic string length
constants and we avoid running over the end of the string should it be
shorter than the prefix.
Signed-off-by: Rene Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
| -rw-r--r-- | builtin/for-each-ref.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/for-each-ref.c b/builtin/for-each-ref.c index 3e1d5c3334..4135980f20 100644 --- a/builtin/for-each-ref.c +++ b/builtin/for-each-ref.c @@ -193,7 +193,7 @@ static int verify_format(const char *format) at = parse_atom(sp + 2, ep); cp = ep + 1; - if (!memcmp(used_atom[at], "color:", 6)) + if (starts_with(used_atom[at], "color:")) need_color_reset_at_eol = !!strcmp(used_atom[at], color_reset); } return 0; |