This simplifies the option parsing. Signed-off-by: Michele Ballabio <barra_cuda@xxxxxxxxxxxx> --- builtin-cat-file.c | 119 ++++++++++++++++++++-------------------------------- 1 files changed, 46 insertions(+), 73 deletions(-) diff --git a/builtin-cat-file.c b/builtin-cat-file.c index b4d0c25..5ef15a4 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|<type>] <sha1> | [--batch|--batch-check] < <list_of_sha1s> ]"; +static const char * const cat_file_usage[] = { + "git-cat-file [-t|-s|-e|-p|<type>] <sha1>", + "git-cat-file [--batch|--batch-check] < <list_of_sha1s>", + 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("<type> 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); } -- 1.5.5.1 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html