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 /fetch-pack.c | |
| parent | bce14aa132e0064d9a9b1c7ad98e71e22c6e0272 (diff) | |
| download | git-50e19a83585bd1aeae6aa01d31e5f783b7ef8b1b.tar.gz | |
Use starts_with() for C strings instead of memcmp()rs/more-starts-with
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 'fetch-pack.c')
| -rw-r--r-- | fetch-pack.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fetch-pack.c b/fetch-pack.c index eeee2bb7e0..b12bd4c59a 100644 --- a/fetch-pack.c +++ b/fetch-pack.c @@ -507,7 +507,7 @@ static void filter_refs(struct fetch_pack_args *args, int keep = 0; next = ref->next; - if (!memcmp(ref->name, "refs/", 5) && + if (starts_with(ref->name, "refs/") && check_refname_format(ref->name, 0)) ; /* trash */ else { |