Skip to content

Commit

Permalink
replace: refactor command-mode determination
Browse files Browse the repository at this point in the history
The git-replace command has three modes: listing, deleting,
and replacing. The first two are selected explicitly. If
none is selected, we fallback to listing when there are no
arguments, and replacing otherwise.

Let's figure out up front which operation we are going to
do, before getting into the application logic. That lets us
simplify our option checks (e.g., we currently have to check
whether a useless "--force" is given both along with an
explicit list, as well as with an implicit one).

This saves some lines, makes the logic easier to follow, and
will facilitate further cleanups.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Jeff King authored and Junio C Hamano committed Apr 29, 2014
1 parent d8779e1 commit 3f495f6
Showing 1 changed file with 6 additions and 8 deletions.
14 changes: 6 additions & 8 deletions builtin/replace.c
Original file line number Diff line number Diff line change
Expand Up @@ -182,12 +182,16 @@ int cmd_replace(int argc, const char **argv, const char *prefix)

argc = parse_options(argc, argv, prefix, options, git_replace_usage, 0);

if (!list && !delete)
if (!argc)
list = 1;

if (list && delete)
usage_msg_opt("-l and -d cannot be used together",
git_replace_usage, options);

if (format && delete)
usage_msg_opt("--format and -d cannot be used together",
if (format && !list)
usage_msg_opt("--format cannot be used when not listing",
git_replace_usage, options);

if (force && (list || delete))
Expand All @@ -207,19 +211,13 @@ int cmd_replace(int argc, const char **argv, const char *prefix)
if (argc != 2)
usage_msg_opt("bad number of arguments",
git_replace_usage, options);
if (format)
usage_msg_opt("--format cannot be used when not listing",
git_replace_usage, options);
return replace_object(argv[0], argv[1], force);
}

/* List refs, even if "list" is not set */
if (argc > 1)
usage_msg_opt("only one pattern can be given with -l",
git_replace_usage, options);
if (force)
usage_msg_opt("-f needs some arguments",
git_replace_usage, options);

return list_replace_refs(argv[0], format);
}

0 comments on commit 3f495f6

Please sign in to comment.