Code

Make pprint_tag void and cleans up call in cmd_cat_file.
authorDavid Rientjes <rientjes@google.com>
Mon, 14 Aug 2006 20:19:15 +0000 (13:19 -0700)
committerJunio C Hamano <junkio@cox.net>
Tue, 15 Aug 2006 01:43:49 +0000 (18:43 -0700)
Signed-off-by: David Rientjes <rientjes@google.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
builtin-cat-file.c

index 814fb0743f7a453114991af6c28bb69ef44d4883..df009ade7aae3ad82974a915efa16525e76ccaed 100644 (file)
@@ -26,7 +26,7 @@ static void flush_buffer(const char *buf, unsigned long size)
        }
 }
 
        }
 }
 
-static int pprint_tag(const unsigned char *sha1, const char *buf, unsigned long size)
+static void pprint_tag(const unsigned char *sha1, const char *buf, unsigned long size)
 {
        /* the parser in tag.c is useless here. */
        const char *endp = buf + size;
 {
        /* the parser in tag.c is useless here. */
        const char *endp = buf + size;
@@ -91,7 +91,6 @@ static int pprint_tag(const unsigned char *sha1, const char *buf, unsigned long
         */
        if (cp < endp)
                flush_buffer(cp, endp - cp);
         */
        if (cp < endp)
                flush_buffer(cp, endp - cp);
-       return 0;
 }
 
 int cmd_cat_file(int argc, const char **argv, const char *prefix)
 }
 
 int cmd_cat_file(int argc, const char **argv, const char *prefix)
@@ -145,8 +144,10 @@ int cmd_cat_file(int argc, const char **argv, const char *prefix)
                buf = read_sha1_file(sha1, type, &size);
                if (!buf)
                        die("Cannot read object %s", argv[2]);
                buf = read_sha1_file(sha1, type, &size);
                if (!buf)
                        die("Cannot read object %s", argv[2]);
-               if (!strcmp(type, tag_type))
-                       return pprint_tag(sha1, buf, size);
+               if (!strcmp(type, tag_type)) {
+                       pprint_tag(sha1, buf, size);
+                       return 0;
+               }
 
                /* otherwise just spit out the data */
                break;
 
                /* otherwise just spit out the data */
                break;