From 9cf71b176cb175589c3ea828f7364893cc45b82f Mon Sep 17 00:00:00 2001 From: Adam Roben Date: Wed, 23 Apr 2008 15:17:44 -0400 Subject: git-cat-file: Small refactor of cmd_cat_file I separated the logic of parsing the arguments from the logic of fetching and outputting the data. cat_one_file now does the latter. Signed-off-by: Adam Roben Signed-off-by: Junio C Hamano --- builtin-cat-file.c | 38 ++++++++++++++++++++++---------------- 1 file changed, 22 insertions(+), 16 deletions(-) (limited to 'builtin-cat-file.c') diff --git a/builtin-cat-file.c b/builtin-cat-file.c index f132d583d3..34a63d1280 100644 --- a/builtin-cat-file.c +++ b/builtin-cat-file.c @@ -76,31 +76,16 @@ static void pprint_tag(const unsigned char *sha1, const char *buf, unsigned long write_or_die(1, cp, endp - cp); } -int cmd_cat_file(int argc, const char **argv, const char *prefix) +static int cat_one_file(int opt, const char *exp_type, const char *obj_name) { unsigned char sha1[20]; enum object_type type; void *buf; unsigned long size; - int opt; - const char *exp_type, *obj_name; - - git_config(git_default_config); - if (argc != 3) - usage("git-cat-file [-t|-s|-e|-p|] "); - exp_type = argv[1]; - obj_name = argv[2]; if (get_sha1(obj_name, sha1)) die("Not a valid object name %s", obj_name); - opt = 0; - if ( exp_type[0] == '-' ) { - opt = exp_type[1]; - if ( !opt || exp_type[2] ) - opt = -1; /* Not a single character option */ - } - buf = NULL; switch (opt) { case 't': @@ -157,3 +142,24 @@ int cmd_cat_file(int argc, const char **argv, const char *prefix) write_or_die(1, buf, size); return 0; } + +int cmd_cat_file(int argc, const char **argv, const char *prefix) +{ + int opt; + const char *exp_type, *obj_name; + + git_config(git_default_config); + if (argc != 3) + usage("git-cat-file [-t|-s|-e|-p|] "); + exp_type = argv[1]; + obj_name = argv[2]; + + opt = 0; + if ( exp_type[0] == '-' ) { + opt = exp_type[1]; + if ( !opt || exp_type[2] ) + opt = -1; /* Not a single character option */ + } + + return cat_one_file(opt, exp_type, obj_name); +} -- cgit v1.2.3 From 4814dbe830baf5a72d8abbbfcc60eeea478b47c2 Mon Sep 17 00:00:00 2001 From: Adam Roben Date: Wed, 23 Apr 2008 15:17:45 -0400 Subject: git-cat-file: Make option parsing a little more flexible This will make it easier to add newer options later. Signed-off-by: Adam Roben Signed-off-by: Junio C Hamano --- builtin-cat-file.c | 42 ++++++++++++++++++++++++++++++------------ 1 file changed, 30 insertions(+), 12 deletions(-) (limited to 'builtin-cat-file.c') diff --git a/builtin-cat-file.c b/builtin-cat-file.c index 34a63d1280..a76bb16b49 100644 --- a/builtin-cat-file.c +++ b/builtin-cat-file.c @@ -143,23 +143,41 @@ static int cat_one_file(int opt, const char *exp_type, const char *obj_name) return 0; } +static const char cat_file_usage[] = "git-cat-file [-t|-s|-e|-p|] "; + int cmd_cat_file(int argc, const char **argv, const char *prefix) { - int opt; - const char *exp_type, *obj_name; + int i, opt = 0; + const char *exp_type = NULL, *obj_name = NULL; git_config(git_default_config); - if (argc != 3) - usage("git-cat-file [-t|-s|-e|-p|] "); - exp_type = argv[1]; - obj_name = argv[2]; - - opt = 0; - if ( exp_type[0] == '-' ) { - opt = exp_type[1]; - if ( !opt || exp_type[2] ) - opt = -1; /* Not a single character option */ + + for (i = 1; i < argc; ++i) { + const char *arg = argv[i]; + + if (!strcmp(arg, "-t") || !strcmp(arg, "-s") || !strcmp(arg, "-e") || !strcmp(arg, "-p")) { + exp_type = arg; + opt = exp_type[1]; + continue; + } + + if (arg[0] == '-') + usage(cat_file_usage); + + if (!exp_type) { + exp_type = arg; + continue; + } + + if (obj_name) + usage(cat_file_usage); + + obj_name = arg; + break; } + if (!exp_type || !obj_name) + usage(cat_file_usage); + return cat_one_file(opt, exp_type, obj_name); } -- cgit v1.2.3 From 05d5667fec9650b049f47edd8cca23a43b135365 Mon Sep 17 00:00:00 2001 From: Adam Roben Date: Wed, 23 Apr 2008 15:17:46 -0400 Subject: git-cat-file: Add --batch-check option This new option allows multiple objects to be specified on stdin. For each object specified, a line of the following form is printed: SP SP LF If the object does not exist in the repository, a line of the following form is printed: SP missing LF Signed-off-by: Adam Roben Signed-off-by: Junio C Hamano --- builtin-cat-file.c | 74 ++++++++++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 72 insertions(+), 2 deletions(-) (limited to 'builtin-cat-file.c') diff --git a/builtin-cat-file.c b/builtin-cat-file.c index a76bb16b49..832cfd184a 100644 --- a/builtin-cat-file.c +++ b/builtin-cat-file.c @@ -143,11 +143,48 @@ static int cat_one_file(int opt, const char *exp_type, const char *obj_name) return 0; } -static const char cat_file_usage[] = "git-cat-file [-t|-s|-e|-p|] "; +static int batch_one_object(const char *obj_name) +{ + unsigned char sha1[20]; + enum object_type type; + unsigned long size; + + if (!obj_name) + return 1; + + if (get_sha1(obj_name, sha1)) { + printf("%s missing\n", obj_name); + return 0; + } + + type = sha1_object_info(sha1, &size); + if (type <= 0) + return 1; + + printf("%s %s %lu\n", sha1_to_hex(sha1), typename(type), size); + + return 0; +} + +static int batch_objects(void) +{ + struct strbuf buf; + + strbuf_init(&buf, 0); + while (strbuf_getline(&buf, stdin, '\n') != EOF) { + int error = batch_one_object(buf.buf); + if (error) + return error; + } + + return 0; +} + +static const char cat_file_usage[] = "git-cat-file [ [-t|-s|-e|-p|] | --batch-check < ]"; int cmd_cat_file(int argc, const char **argv, const char *prefix) { - int i, opt = 0; + int i, opt = 0, batch_check = 0; const char *exp_type = NULL, *obj_name = NULL; git_config(git_default_config); @@ -155,7 +192,28 @@ int cmd_cat_file(int argc, const char **argv, const char *prefix) for (i = 1; i < argc; ++i) { const char *arg = argv[i]; + if (!strcmp(arg, "--batch-check")) { + if (opt) { + error("git-cat-file: Can't use --batch-check with -%c", opt); + usage(cat_file_usage); + } else if (exp_type) { + error("git-cat-file: Can't use --batch-check when a type (\"%s\") is specified", exp_type); + usage(cat_file_usage); + } else if (obj_name) { + error("git-cat-file: Can't use --batch-check when an object (\"%s\") is specified", obj_name); + usage(cat_file_usage); + } + + batch_check = 1; + continue; + } + if (!strcmp(arg, "-t") || !strcmp(arg, "-s") || !strcmp(arg, "-e") || !strcmp(arg, "-p")) { + if (batch_check) { + error("git-cat-file: Can't use %s with --batch-check", arg); + usage(cat_file_usage); + } + exp_type = arg; opt = exp_type[1]; continue; @@ -165,6 +223,11 @@ int cmd_cat_file(int argc, const char **argv, const char *prefix) usage(cat_file_usage); if (!exp_type) { + if (batch_check) { + error("git-cat-file: Can't specify a type (\"%s\") with --batch-check", arg); + usage(cat_file_usage); + } + exp_type = arg; continue; } @@ -172,10 +235,17 @@ int cmd_cat_file(int argc, const char **argv, const char *prefix) if (obj_name) usage(cat_file_usage); + // We should have hit one of the earlier if (batch_check) cases before + // getting here. + assert(!batch_check); + obj_name = arg; break; } + if (batch_check) + return batch_objects(); + if (!exp_type || !obj_name) usage(cat_file_usage); -- cgit v1.2.3 From a8128ed62858063e29edc066b14b8b0fa6257cc2 Mon Sep 17 00:00:00 2001 From: Adam Roben Date: Wed, 23 Apr 2008 15:17:47 -0400 Subject: git-cat-file: Add --batch option --batch is similar to --batch-check, except that the contents of each object is also printed. The output's form is: SP SP LF LF Signed-off-by: Adam Roben Signed-off-by: Junio C Hamano --- builtin-cat-file.c | 63 ++++++++++++++++++++++++++++++++++++++---------------- 1 file changed, 45 insertions(+), 18 deletions(-) (limited to 'builtin-cat-file.c') diff --git a/builtin-cat-file.c b/builtin-cat-file.c index 832cfd184a..b4d0c2545f 100644 --- a/builtin-cat-file.c +++ b/builtin-cat-file.c @@ -143,11 +143,12 @@ static int cat_one_file(int opt, const char *exp_type, const char *obj_name) return 0; } -static int batch_one_object(const char *obj_name) +static int batch_one_object(const char *obj_name, int print_contents) { unsigned char sha1[20]; enum object_type type; unsigned long size; + void *contents = contents; if (!obj_name) return 1; @@ -157,22 +158,33 @@ static int batch_one_object(const char *obj_name) return 0; } - type = sha1_object_info(sha1, &size); + if (print_contents) + contents = read_sha1_file(sha1, &type, &size); + else + type = sha1_object_info(sha1, &size); + if (type <= 0) return 1; printf("%s %s %lu\n", sha1_to_hex(sha1), typename(type), size); + fflush(stdout); + + if (print_contents) { + write_or_die(1, contents, size); + printf("\n"); + fflush(stdout); + } return 0; } -static int batch_objects(void) +static int batch_objects(int print_contents) { struct strbuf buf; strbuf_init(&buf, 0); while (strbuf_getline(&buf, stdin, '\n') != EOF) { - int error = batch_one_object(buf.buf); + int error = batch_one_object(buf.buf, print_contents); if (error) return error; } @@ -180,37 +192,51 @@ static int batch_objects(void) return 0; } -static const char cat_file_usage[] = "git-cat-file [ [-t|-s|-e|-p|] | --batch-check < ]"; +static const char cat_file_usage[] = "git-cat-file [ [-t|-s|-e|-p|] | [--batch|--batch-check] < ]"; int cmd_cat_file(int argc, const char **argv, const char *prefix) { - int i, opt = 0, batch_check = 0; + int i, opt = 0, batch = 0, batch_check = 0; const char *exp_type = NULL, *obj_name = NULL; git_config(git_default_config); for (i = 1; i < argc; ++i) { const char *arg = argv[i]; + int is_batch = 0, is_batch_check = 0; + + is_batch = !strcmp(arg, "--batch"); + if (!is_batch) + is_batch_check = !strcmp(arg, "--batch-check"); - if (!strcmp(arg, "--batch-check")) { + if (is_batch || is_batch_check) { if (opt) { - error("git-cat-file: Can't use --batch-check with -%c", opt); + error("git-cat-file: Can't use %s with -%c", arg, opt); usage(cat_file_usage); } else if (exp_type) { - error("git-cat-file: Can't use --batch-check when a type (\"%s\") is specified", exp_type); + error("git-cat-file: Can't use %s when a type (\"%s\") is specified", arg, exp_type); usage(cat_file_usage); } else if (obj_name) { - error("git-cat-file: Can't use --batch-check when an object (\"%s\") is specified", obj_name); + error("git-cat-file: Can't use %s when an object (\"%s\") is specified", arg, obj_name); usage(cat_file_usage); } - batch_check = 1; + if ((is_batch && batch_check) || (is_batch_check && batch)) { + error("git-cat-file: Can't use %s with %s", arg, is_batch ? "--batch-check" : "--batch"); + usage(cat_file_usage); + } + + if (is_batch) + batch = 1; + else + batch_check = 1; + continue; } if (!strcmp(arg, "-t") || !strcmp(arg, "-s") || !strcmp(arg, "-e") || !strcmp(arg, "-p")) { - if (batch_check) { - error("git-cat-file: Can't use %s with --batch-check", arg); + if (batch || batch_check) { + error("git-cat-file: Can't use %s with %s", arg, batch ? "--batch" : "--batch-check"); usage(cat_file_usage); } @@ -223,8 +249,8 @@ int cmd_cat_file(int argc, const char **argv, const char *prefix) usage(cat_file_usage); if (!exp_type) { - if (batch_check) { - error("git-cat-file: Can't specify a type (\"%s\") with --batch-check", arg); + if (batch || batch_check) { + error("git-cat-file: Can't specify a type (\"%s\") with %s", arg, batch ? "--batch" : "--batch-check"); usage(cat_file_usage); } @@ -235,16 +261,17 @@ int cmd_cat_file(int argc, const char **argv, const char *prefix) if (obj_name) usage(cat_file_usage); - // We should have hit one of the earlier if (batch_check) cases before + // We should have hit one of the earlier if (batch || batch_check) cases before // getting here. + assert(!batch); assert(!batch_check); obj_name = arg; break; } - if (batch_check) - return batch_objects(); + if (batch || batch_check) + return batch_objects(batch); if (!exp_type || !obj_name) usage(cat_file_usage); -- cgit v1.2.3 From 15d8e5651948e3914f4b765d7d1dc1bc2f5c4073 Mon Sep 17 00:00:00 2001 From: Michele Ballabio Date: Fri, 23 May 2008 16:19:42 +0200 Subject: builtin-cat-file.c: use parse_options() This simplifies the option parsing. Signed-off-by: Michele Ballabio Signed-off-by: Junio C Hamano --- builtin-cat-file.c | 119 +++++++++++++++++++++-------------------------------- 1 file changed, 46 insertions(+), 73 deletions(-) (limited to 'builtin-cat-file.c') diff --git a/builtin-cat-file.c b/builtin-cat-file.c index b4d0c2545f..5ef15a4fa9 100644 --- a/builtin-cat-file.c +++ b/builtin-cat-file.c @@ -8,6 +8,10 @@ #include "tag.h" #include "tree.h" #include "builtin.h" +#include "parse-options.h" + +#define BATCH 1 +#define BATCH_CHECK 2 static void pprint_tag(const unsigned char *sha1, const char *buf, unsigned long size) { @@ -158,7 +162,7 @@ static int batch_one_object(const char *obj_name, int print_contents) return 0; } - if (print_contents) + if (print_contents == BATCH) contents = read_sha1_file(sha1, &type, &size); else type = sha1_object_info(sha1, &size); @@ -169,7 +173,7 @@ static int batch_one_object(const char *obj_name, int print_contents) printf("%s %s %lu\n", sha1_to_hex(sha1), typename(type), size); fflush(stdout); - if (print_contents) { + if (print_contents == BATCH) { write_or_die(1, contents, size); printf("\n"); fflush(stdout); @@ -192,89 +196,58 @@ static int batch_objects(int print_contents) return 0; } -static const char cat_file_usage[] = "git-cat-file [ [-t|-s|-e|-p|] | [--batch|--batch-check] < ]"; +static const char * const cat_file_usage[] = { + "git-cat-file [-t|-s|-e|-p|] ", + "git-cat-file [--batch|--batch-check] < ", + NULL +}; int cmd_cat_file(int argc, const char **argv, const char *prefix) { - int i, opt = 0, batch = 0, batch_check = 0; + int opt = 0, batch = 0; const char *exp_type = NULL, *obj_name = NULL; - git_config(git_default_config); - - for (i = 1; i < argc; ++i) { - const char *arg = argv[i]; - int is_batch = 0, is_batch_check = 0; - - is_batch = !strcmp(arg, "--batch"); - if (!is_batch) - is_batch_check = !strcmp(arg, "--batch-check"); - - if (is_batch || is_batch_check) { - if (opt) { - error("git-cat-file: Can't use %s with -%c", arg, opt); - usage(cat_file_usage); - } else if (exp_type) { - error("git-cat-file: Can't use %s when a type (\"%s\") is specified", arg, exp_type); - usage(cat_file_usage); - } else if (obj_name) { - error("git-cat-file: Can't use %s when an object (\"%s\") is specified", arg, obj_name); - usage(cat_file_usage); - } - - if ((is_batch && batch_check) || (is_batch_check && batch)) { - error("git-cat-file: Can't use %s with %s", arg, is_batch ? "--batch-check" : "--batch"); - usage(cat_file_usage); - } - - if (is_batch) - batch = 1; - else - batch_check = 1; + const struct option options[] = { + OPT_GROUP(" can be one of: blob, tree, commit, tag"), + OPT_SET_INT('t', NULL, &opt, "show object type", 't'), + OPT_SET_INT('s', NULL, &opt, "show object size", 's'), + OPT_SET_INT('e', NULL, &opt, + "exit with zero when there's no error", 'e'), + OPT_SET_INT('p', NULL, &opt, "pretty-print object's content", 'p'), + OPT_SET_INT(0, "batch", &batch, + "show info and content of objects feeded on stdin", BATCH), + OPT_SET_INT(0, "batch-check", &batch, + "show info about objects feeded on stdin", + BATCH_CHECK), + OPT_END() + }; - continue; - } - - if (!strcmp(arg, "-t") || !strcmp(arg, "-s") || !strcmp(arg, "-e") || !strcmp(arg, "-p")) { - if (batch || batch_check) { - error("git-cat-file: Can't use %s with %s", arg, batch ? "--batch" : "--batch-check"); - usage(cat_file_usage); - } - - exp_type = arg; - opt = exp_type[1]; - continue; - } - - if (arg[0] == '-') - usage(cat_file_usage); - - if (!exp_type) { - if (batch || batch_check) { - error("git-cat-file: Can't specify a type (\"%s\") with %s", arg, batch ? "--batch" : "--batch-check"); - usage(cat_file_usage); - } - - exp_type = arg; - continue; - } + git_config(git_default_config); - if (obj_name) - usage(cat_file_usage); + if (argc != 3 && argc != 2) + usage_with_options(cat_file_usage, options); - // We should have hit one of the earlier if (batch || batch_check) cases before - // getting here. - assert(!batch); - assert(!batch_check); + argc = parse_options(argc, argv, options, cat_file_usage, 0); - obj_name = arg; - break; + if (opt) { + if (argc == 1) + obj_name = argv[0]; + else + usage_with_options(cat_file_usage, options); + } + if (!opt && !batch) { + if (argc == 2) { + exp_type = argv[0]; + obj_name = argv[1]; + } else + usage_with_options(cat_file_usage, options); + } + if (batch && (opt || argc)) { + usage_with_options(cat_file_usage, options); } - if (batch || batch_check) + if (batch) return batch_objects(batch); - if (!exp_type || !obj_name) - usage(cat_file_usage); - return cat_one_file(opt, exp_type, obj_name); } -- cgit v1.2.3