Remove redundant code, eliminate one static variable

Signed-off-by: Heikki Orsila <heikki.orsila@iki.fi>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/sha1_file.c b/sha1_file.c
index 141f5a713a2e542c0bcccbfa57b0dccc3142f362..c3170961cedd080f7021093b50e3d607169874bd 100644 (file)
--- a/sha1_file.c
+++ b/sha1_file.c
@@ -176,21 +176,22 @@ char *sha1_file_name(const unsigned char *sha1)
        return base;
 }
 
        return base;
 }
 

-char *sha1_pack_name(const unsigned char *sha1)
+static char *sha1_get_pack_name(const unsigned char *sha1,
+                               char **name, char **base)

 {
        static const char hex[] = "0123456789abcdef";
 {
        static const char hex[] = "0123456789abcdef";

-       static char *name, *base, *buf;
+       char *buf;

        int i;
 
        int i;
 

-       if (!base) {
+       if (!*base) {

                const char *sha1_file_directory = get_object_directory();
                int len = strlen(sha1_file_directory);
                const char *sha1_file_directory = get_object_directory();
                int len = strlen(sha1_file_directory);

-               base = xmalloc(len + 60);
-               sprintf(base, "%s/pack/pack-1234567890123456789012345678901234567890.pack", sha1_file_directory);
-               name = base + len + 11;
+               *base = xmalloc(len + 60);
+               sprintf(*base, "%s/pack/pack-1234567890123456789012345678901234567890.pack", sha1_file_directory);
+               *name = *base + len + 11;

        }
 
        }
 

-       buf = name;
+       buf = *name;

 
        for (i = 0; i < 20; i++) {
                unsigned int val = *sha1++;
 
        for (i = 0; i < 20; i++) {
                unsigned int val = *sha1++;


@@ -198,32 +199,21 @@ char *sha1_pack_name(const unsigned char *sha1)
                *buf++ = hex[val & 0xf];
        }
 
                *buf++ = hex[val & 0xf];
        }
 

-       return base;
+       return *base;

 }
 
 }
 

-char *sha1_pack_index_name(const unsigned char *sha1)
+char *sha1_pack_name(const unsigned char *sha1)

 {
 {

-       static const char hex[] = "0123456789abcdef";
-       static char *name, *base, *buf;
-       int i;
-
-       if (!base) {
-               const char *sha1_file_directory = get_object_directory();
-               int len = strlen(sha1_file_directory);
-               base = xmalloc(len + 60);
-               sprintf(base, "%s/pack/pack-1234567890123456789012345678901234567890.idx", sha1_file_directory);
-               name = base + len + 11;
-       }
+       static char *name, *base;

 
 

-       buf = name;
+       return sha1_get_pack_name(sha1, &name, &base);
+}

 
 

-       for (i = 0; i < 20; i++) {
-               unsigned int val = *sha1++;
-               *buf++ = hex[val >> 4];
-               *buf++ = hex[val & 0xf];
-       }
+char *sha1_pack_index_name(const unsigned char *sha1)
+{
+       static char *name, *base;

 
 

-       return base;
+       return sha1_get_pack_name(sha1, &name, &base);

 }
 
 struct alternate_object_database *alt_odb_list;
 }
 
 struct alternate_object_database *alt_odb_list;