OSDN Git Service

drop length argument of has_extension
authorRene Scharfe <rene.scharfe@lsrfire.ath.cx>
Fri, 11 Aug 2006 12:01:45 +0000 (14:01 +0200)
committerJunio C Hamano <junkio@cox.net>
Fri, 11 Aug 2006 23:06:34 +0000 (16:06 -0700)
As Fredrik points out the current interface of has_extension() is
potentially confusing.  Its parameters include both a nul-terminated
string and a length-limited string.

This patch drops the length argument, requiring two nul-terminated
strings; all callsites are updated.  I checked that all of them indeed
provide nul-terminated strings.  Filenames need to be nul-terminated
anyway if they are to be passed to open() etc.  The performance penalty
of the additional strlen() is negligible compared to the system calls
which inevitably surround has_extension() calls.

Additionally, change has_extension() to use size_t inside instead of
int, as that is the exact type strlen() returns and memcmp() expects.

Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Signed-off-by: Junio C Hamano <junkio@cox.net>
builtin-help.c
git-compat-util.h
http-fetch.c
index-pack.c
local-fetch.c
refs.c
sha1_file.c
verify-pack.c

index 7a7f775..6484cb9 100644 (file)
@@ -140,7 +140,7 @@ static void list_commands(const char *exec_path, const char *pattern)
                        continue;
 
                entlen = strlen(de->d_name);
-               if (has_extension(de->d_name, entlen, ".exe"))
+               if (has_extension(de->d_name, ".exe"))
                        entlen -= 4;
 
                if (longest < entlen)
index dd92093..b2e1895 100644 (file)
@@ -139,9 +139,10 @@ static inline ssize_t xwrite(int fd, const void *buf, size_t len)
        }
 }
 
-static inline int has_extension(const char *filename, int len, const char *ext)
+static inline int has_extension(const char *filename, const char *ext)
 {
-       int extlen = strlen(ext);
+       size_t len = strlen(filename);
+       size_t extlen = strlen(ext);
        return len > extlen && !memcmp(filename + len - extlen, ext, extlen);
 }
 
index 6ea39f0..de5fc44 100644 (file)
@@ -870,7 +870,7 @@ static void process_ls_pack(struct remote_ls_ctx *ls)
 
        if (strlen(ls->dentry_name) == 63 &&
            !strncmp(ls->dentry_name, "objects/pack/pack-", 18) &&
-           has_extension(ls->dentry_name, 63, ".pack")) {
+           has_extension(ls->dentry_name, ".pack")) {
                get_sha1_hex(ls->dentry_name + 18, sha1);
                setup_index(ls->repo, sha1);
        }
index a91e39e..b20659c 100644 (file)
@@ -447,7 +447,7 @@ int main(int argc, char **argv)
                usage(index_pack_usage);
        if (!index_name) {
                int len = strlen(pack_name);
-               if (!has_extension(pack_name, len, ".pack"))
+               if (!has_extension(pack_name, ".pack"))
                        die("packfile name '%s' does not end with '.pack'",
                            pack_name);
                index_name_buf = xmalloc(len);
index b6ec170..7d01845 100644 (file)
@@ -44,7 +44,7 @@ static int setup_indices(void)
        while ((de = readdir(dir)) != NULL) {
                int namelen = strlen(de->d_name);
                if (namelen != 50 ||
-                   !has_extension(de->d_name, namelen, ".pack"))
+                   !has_extension(de->d_name, ".pack"))
                        continue;
                get_sha1_hex(de->d_name + 5, sha1);
                setup_index(sha1);
diff --git a/refs.c b/refs.c
index b01835f..28a9394 100644 (file)
--- a/refs.c
+++ b/refs.c
@@ -147,7 +147,7 @@ static int do_for_each_ref(const char *base, int (*fn)(const char *path, const u
                        namelen = strlen(de->d_name);
                        if (namelen > 255)
                                continue;
-                       if (has_extension(de->d_name, namelen, ".lock"))
+                       if (has_extension(de->d_name, ".lock"))
                                continue;
                        memcpy(path + baselen, de->d_name, namelen+1);
                        if (stat(git_path("%s", path), &st) < 0)
index a1bb01c..3db956d 100644 (file)
@@ -590,7 +590,7 @@ static void prepare_packed_git_one(char *objdir, int local)
                int namelen = strlen(de->d_name);
                struct packed_git *p;
 
-               if (!has_extension(de->d_name, namelen, ".idx"))
+               if (!has_extension(de->d_name, ".idx"))
                        continue;
 
                /* we have .idx.  Is it a file we can map? */
index f440a39..357970d 100644 (file)
@@ -16,10 +16,10 @@ static int verify_one_pack(const char *path, int verbose)
         * In addition to "foo.idx" we accept "foo.pack" and "foo";
         * normalize these forms to "foo.idx" for add_packed_git().
         */
-       if (has_extension(arg, len, ".pack")) {
+       if (has_extension(arg, ".pack")) {
                strcpy(arg + len - 5, ".idx");
                len--;
-       } else if (!has_extension(arg, len, ".idx")) {
+       } else if (!has_extension(arg, ".idx")) {
                if (len + 4 >= PATH_MAX)
                        return error("name too long: %s.idx", arg);
                strcpy(arg + len, ".idx");