]> git.ipfire.org Git - thirdparty/git.git/commitdiff
git.c: improve code readability in cmd_main()
authorDaniel Sonbolian <dsal3389@gmail.com>
Sat, 8 Oct 2022 16:21:37 +0000 (16:21 +0000)
committerJunio C Hamano <gitster@pobox.com>
Sun, 9 Oct 2022 05:11:37 +0000 (22:11 -0700)
Check for an error condition whose body unconditionally exists
first, and then perform the special casing of "version" and "help"
as part of the preparation for the "normal codepath".  This makes
the code simpler to read.

Signed-off-by: Daniel Sonbolian <dsal3389@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git.c

diff --git a/git.c b/git.c
index d7a7a82008b41e80c17176a9865ff49bf538031a..a10fe447b1f6bceba1e7b33a77f80128010fdd2f 100644 (file)
--- a/git.c
+++ b/git.c
@@ -893,12 +893,8 @@ int cmd_main(int argc, const char **argv)
        argv++;
        argc--;
        handle_options(&argv, &argc, NULL);
-       if (argc > 0) {
-               if (!strcmp("--version", argv[0]) || !strcmp("-v", argv[0]))
-                       argv[0] = "version";
-               else if (!strcmp("--help", argv[0]) || !strcmp("-h", argv[0]))
-                       argv[0] = "help";
-       } else {
+
+       if (!argc) {
                /* The user didn't specify a command; give them help */
                commit_pager_choice();
                printf(_("usage: %s\n\n"), git_usage_string);
@@ -906,6 +902,12 @@ int cmd_main(int argc, const char **argv)
                printf("\n%s\n", _(git_more_info_string));
                exit(1);
        }
+
+       if (!strcmp("--version", argv[0]) || !strcmp("-v", argv[0]))
+               argv[0] = "version";
+       else if (!strcmp("--help", argv[0]) || !strcmp("-h", argv[0]))
+               argv[0] = "help";
+
        cmd = argv[0];
 
        /*