diff options
| author | Jeff King <peff@peff.net> | 2014-06-30 16:58:25 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2014-06-30 20:43:16 (GMT) |
| commit | 2975c770ca609ea5afc80631c4ac9087c527b6fd (patch) | |
| tree | 54d7b59ce331c9e2079ad730a311347224f76d59 /sha1_file.c | |
| parent | f52a35fd63cc6d570083cedc15576d01c0968d98 (diff) | |
| download | git-2975c770ca609ea5afc80631c4ac9087c527b6fd.zip git-2975c770ca609ea5afc80631c4ac9087c527b6fd.tar.gz git-2975c770ca609ea5afc80631c4ac9087c527b6fd.tar.bz2 | |
replace has_extension with ends_with
These two are almost the same function, with the exception
that has_extension only matches if there is content before
the suffix. So ends_with(".exe", ".exe") is true, but
has_extension would not be.
This distinction does not matter to any of the callers,
though, and we can just replace uses of has_extension with
ends_with. We prefer the "ends_with" name because it is more
generic, and there is nothing about the function that
requires it to be used for file extensions.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'sha1_file.c')
| -rw-r--r-- | sha1_file.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/sha1_file.c b/sha1_file.c index 394fa45..93b794f 100644 --- a/sha1_file.c +++ b/sha1_file.c @@ -1204,7 +1204,7 @@ static void prepare_packed_git_one(char *objdir, int local) strbuf_setlen(&path, dirnamelen); strbuf_addstr(&path, de->d_name); - if (has_extension(de->d_name, ".idx")) { + if (ends_with(de->d_name, ".idx")) { /* Don't reopen a pack we already have. */ for (p = packed_git; p; p = p->next) { if (!memcmp(path.buf, p->pack_name, path.len - 4)) @@ -1222,10 +1222,10 @@ static void prepare_packed_git_one(char *objdir, int local) if (!report_garbage) continue; - if (has_extension(de->d_name, ".idx") || - has_extension(de->d_name, ".pack") || - has_extension(de->d_name, ".bitmap") || - has_extension(de->d_name, ".keep")) + if (ends_with(de->d_name, ".idx") || + ends_with(de->d_name, ".pack") || + ends_with(de->d_name, ".bitmap") || + ends_with(de->d_name, ".keep")) string_list_append(&garbage, path.buf); else report_garbage("garbage found", path.buf); |