[PATCH] Header files for object parsing

[git.git] / show-diff.c

diff --git a/show-diff.c b/show-diff.c
index 4221d3a..8a66e59 100644 (file)
--- a/show-diff.c
+++ b/show-diff.c
@@ -5,13 +5,68 @@
  */
 #include "cache.h"
 
-static void show_differences(char *name,
-       void *old_contents, unsigned long long old_size)
+static char *diff_cmd = "diff -L '%s' -u -N  - '%s'";
+
+/* Help to copy the thing properly quoted for the shell safety.
+ * any single quote is replaced with '\'', and the caller is
+ * expected to enclose the result within a single quote pair.
+ *
+ * E.g.
+ *  original     sq_expand     result
+ *  name     ==> name      ==> 'name'
+ *  a b      ==> a b       ==> 'a b'
+ *  a'b      ==> a'\''b    ==> 'a'\''b'
+ *
+ * NOTE! The returned memory belongs to this function so
+ * do not free it.
+ */
+static char *sq_expand(char *src)
+{
+       static char *buf = NULL;
+       static int buf_size = -1;
+       int cnt, c;
+       char *cp;
+
+       /* count single quote characters */ 
+       for (cnt = 0, cp = src; *cp; cnt++, cp++)
+               if (*cp == '\'')
+                       cnt += 3;
+
+       if (buf_size < cnt) {
+               free(buf);
+               buf_size = cnt;
+               buf = malloc(cnt);
+       }
+
+       cp = buf;
+       while ((c = *src++)) {
+               if (c != '\'')
+                       *cp++ = c;
+               else {
+                       cp = strcpy(cp, "'\\''");
+                       cp += 4;
+               }
+       }
+       *cp = 0;
+       return buf;
+}
+
+static void show_differences(char *name, void *old_contents,
+                            unsigned long long old_size)
 {
-       static char cmd[1000];
        FILE *f;
+       static char *cmd = NULL;
+       static int cmd_size = -1;
+
+       char *name_sq = sq_expand(name);
+       int cmd_required_length = strlen(name_sq) * 2 + strlen(diff_cmd);
 
-       snprintf(cmd, sizeof(cmd), "diff -L %s -u -N  - %s", name, name);
+       if (cmd_size < cmd_required_length) {
+               free(cmd);
+               cmd_size = cmd_required_length;
+               cmd = malloc(cmd_required_length);
+       }
+       snprintf(cmd, cmd_size, diff_cmd, name_sq, name_sq);
        f = popen(cmd, "w");
        if (old_size)
                fwrite(old_contents, old_size, 1, f);
@@ -55,7 +110,7 @@ static void show_diff_empty(struct cache_entry *ce)
        }
 }
 
-static const char *show_diff_usage = "show-diff [-s] [-q] [-z] [paths...]";
+static const char *show_diff_usage = "show-diff [-q] [-s] [-z] [paths...]";
 
 static int matches_pathspec(struct cache_entry *ce, char **spec, int cnt)
 {
@@ -85,9 +140,8 @@ int main(int argc, char **argv)
                        silent_on_nonexisting_files = silent = 1;
                else if (!strcmp(argv[1], "-q"))
                        silent_on_nonexisting_files = 1;
-               else if (!strcmp(argv[1], "-z")) {
+               else if (!strcmp(argv[1], "-z"))
                        machine_readable = 1;
-               }
                else
                        usage(show_diff_usage);
                argv++; argc--;
@@ -106,12 +160,25 @@ int main(int argc, char **argv)
                int changed;
                unsigned long size;
                char type[20];
-               void *new;
+               void *old;
 
-               if (1 <argc &&
+               if (1 < argc &&
                    ! matches_pathspec(ce, argv+1, argc-1))
                        continue;
 
+               if (ce_stage(ce)) {
+                       if (machine_readable)
+                               printf("U %s%c", ce->name, 0);
+                       else
+                               printf("%s: Unmerged\n",
+                                      ce->name);
+                       while (i < entries &&
+                              !strcmp(ce->name, active_cache[i]->name))
+                               i++;
+                       i--; /* compensate for loop control increments */
+                       continue;
+               }
+ 
                if (stat(ce->name, &st) < 0) {
                        if (errno == ENOENT && silent_on_nonexisting_files)
                                continue;
@@ -137,9 +204,9 @@ int main(int argc, char **argv)
                if (silent)
                        continue;
 
-               new = read_sha1_file(ce->sha1, type, &size);
-               show_differences(ce->name, new, size);
-               free(new);
+               old = read_sha1_file(ce->sha1, type, &size);
+               show_differences(ce->name, old, size);
+               free(old);
        }
        return 0;
 }