cat-file: use streaming API to print blobs

Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/builtin/cat-file.c b/builtin/cat-file.c
index 8ed501f220424976cc30f4a4dbf3d59f979902be..36a9104433e23422aab39b1912e998a7f54cd3f4 100644 (file)
--- a/builtin/cat-file.c
+++ b/builtin/cat-file.c
@@ -11,6 +11,7 @@
 #include "parse-options.h"
 #include "diff.h"
 #include "userdiff.h"
 #include "parse-options.h"
 #include "diff.h"
 #include "userdiff.h"

+#include "streaming.h"

 
 #define BATCH 1
 #define BATCH_CHECK 2
 
 #define BATCH 1
 #define BATCH_CHECK 2


@@ -127,6 +128,8 @@ static int cat_one_file(int opt, const char *exp_type, const char *obj_name)
                        return cmd_ls_tree(2, ls_args, NULL);
                }
 
                        return cmd_ls_tree(2, ls_args, NULL);
                }
 

+               if (type == OBJ_BLOB)
+                       return stream_blob_to_fd(1, sha1, NULL, 0);

                buf = read_sha1_file(sha1, &type, &size);
                if (!buf)
                        die("Cannot read object %s", obj_name);
                buf = read_sha1_file(sha1, &type, &size);
                if (!buf)
                        die("Cannot read object %s", obj_name);


@@ -149,6 +152,28 @@ static int cat_one_file(int opt, const char *exp_type, const char *obj_name)
                break;
 
        case 0:
                break;
 
        case 0:

+               if (type_from_string(exp_type) == OBJ_BLOB) {
+                       unsigned char blob_sha1[20];
+                       if (sha1_object_info(sha1, NULL) == OBJ_TAG) {
+                               enum object_type type;
+                               unsigned long size;
+                               char *buffer = read_sha1_file(sha1, &type, &size);
+                               if (memcmp(buffer, "object ", 7) ||
+                                   get_sha1_hex(buffer + 7, blob_sha1))
+                                       die("%s not a valid tag", sha1_to_hex(sha1));
+                               free(buffer);
+                       } else
+                               hashcpy(blob_sha1, sha1);
+
+                       if (sha1_object_info(blob_sha1, NULL) == OBJ_BLOB)
+                               return stream_blob_to_fd(1, blob_sha1, NULL, 0);
+                       /*
+                        * we attempted to dereference a tag to a blob
+                        * and failed; there may be new dereference
+                        * mechanisms this code is not aware of.
+                        * fall-back to the usual case.
+                        */
+               }

                buf = read_object_with_reference(sha1, exp_type, &size, NULL);
                break;
 
                buf = read_object_with_reference(sha1, exp_type, &size, NULL);
                break;
 

diff --git a/t/t1050-large.sh b/t/t1050-large.sh
index ded66b3228920220c710788337874f46401e83a1..f662fefa6740ccea0d2a60621c43f1cc7262dfc3 100755 (executable)
--- a/t/t1050-large.sh
+++ b/t/t1050-large.sh
@@ -116,11 +116,11 @@ test_expect_success 'hash-object' '
        git hash-object large1
 '
 
        git hash-object large1
 '
 

-test_expect_failure 'cat-file a large file' '
+test_expect_success 'cat-file a large file' '

        git cat-file blob :large1 >/dev/null
 '
 
        git cat-file blob :large1 >/dev/null
 '
 

-test_expect_failure 'cat-file a large file from a tag' '
+test_expect_success 'cat-file a large file from a tag' '

        git tag -m largefile largefiletag :large1 &&
        git cat-file blob largefiletag >/dev/null
 '
        git tag -m largefile largefiletag :large1 &&
        git cat-file blob largefiletag >/dev/null
 '