diff-index.c: unify handling of command line paths
authorJunio C Hamano <gitster@pobox.com>
Wed, 27 Jun 2012 19:05:52 +0000 (12:05 -0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 28 Jun 2012 23:09:40 +0000 (16:09 -0700)
Regardless of where in the directory hierarchy you are, "-" on the
command line means the standard input. The old code knew too much
about how the low level machinery uses paths to read from the
working tree and did not bother to have the same check for "-" when
the command is run from the top-level.

Unify the codepaths for subdirectory case and toplevel case into one
and make it clearer.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
diff-no-index.c
index 31cc5b187e5949cdcb484876cf49db4ebdd31c50..ac41b68b0a121b8068cae78ddd328387fb11ce0f 100644 (file)
@@ -170,7 +170,7 @@ void diff_no_index(struct rev_info *revs,
                   int argc, const char **argv,
                   int nongit, const char *prefix)
 {
-       int i;
+       int i, prefixlen;
        int no_index = 0;
        unsigned options = 0;
        const char *paths[2];
@@ -230,23 +230,18 @@ void diff_no_index(struct rev_info *revs,
        if (!DIFF_OPT_TST(&revs->diffopt, EXIT_WITH_STATUS))
                setup_pager();
 
-       if (prefix) {
-               int len = strlen(prefix);
-
-               for (i = 0; i < 2; i++) {
-                       const char *p = argv[argc - 2 + i];
+       prefixlen = prefix ? strlen(prefix) : 0;
+       for (i = 0; i < 2; i++) {
+               const char *p = argv[argc - 2 + i];
+               if (!strcmp(p, "-"))
                        /*
-                        * stdin should be spelled as '-'; if you have
-                        * path that is '-', spell it as ./-.
+                        * stdin should be spelled as "-"; if you have
+                        * path that is "-", spell it as "./-".
                         */
-                       p = (strcmp(p, "-")
-                            ? xstrdup(prefix_filename(prefix, len, p))
-                            : p);
-                       paths[i] = p;
-               }
-       } else {
-               for (i = 0; i < 2; i++)
-                       paths[i] = argv[argc - 2 + i];
+                       p = p;
+               else if (prefixlen)
+                       p = xstrdup(prefix_filename(prefix, prefixlen, p));
+               paths[i] = p;
        }
        revs->diffopt.skip_stat_unmatch = 1;
        if (!revs->diffopt.output_format)