Merge branch 'ew/pretty-fmt' into next
authorJunio C Hamano <junkio@cox.net>
Wed, 17 May 2006 10:04:14 +0000 (03:04 -0700)
committerJunio C Hamano <junkio@cox.net>
Wed, 17 May 2006 10:04:14 +0000 (03:04 -0700)
* ew/pretty-fmt:
commit: allow --pretty= args to be abbreviated

Conflicts:

commit.c - adjust to --pretty=email

1  2 
commit.c
diff --combined commit.c
index 93b3903ea78fac0b5a40074ec64d0c98bb8cfa94,4a26070c137243bf099cf10415b2910eca230368..84558bac29afd64e295a451ad7baa099237ae045
+++ b/commit.c
@@@ -22,25 -22,33 +22,34 @@@ struct sort_nod
  
  const char *commit_type = "commit";
  
+ struct cmt_fmt_map {
+       const char *n;
+       size_t cmp_len;
+       enum cmit_fmt v;
+ } cmt_fmts[] = {
+       { "raw",        1,      CMIT_FMT_RAW },
+       { "medium",     1,      CMIT_FMT_MEDIUM },
+       { "short",      1,      CMIT_FMT_SHORT },
++      { "email",      1,      CMIT_FMT_EMAIL },
+       { "full",       5,      CMIT_FMT_FULL },
+       { "fuller",     5,      CMIT_FMT_FULLER },
+       { "oneline",    1,      CMIT_FMT_ONELINE },
+ };
  enum cmit_fmt get_commit_format(const char *arg)
  {
-       if (!*arg)
+       int i;
+       if (!arg || !*arg)
                return CMIT_FMT_DEFAULT;
-       if (!strcmp(arg, "=raw"))
-               return CMIT_FMT_RAW;
-       if (!strcmp(arg, "=medium"))
-               return CMIT_FMT_MEDIUM;
-       if (!strcmp(arg, "=short"))
-               return CMIT_FMT_SHORT;
-       if (!strcmp(arg, "=full"))
-               return CMIT_FMT_FULL;
-       if (!strcmp(arg, "=fuller"))
-               return CMIT_FMT_FULLER;
-       if (!strcmp(arg, "=email"))
-               return CMIT_FMT_EMAIL;
-       if (!strcmp(arg, "=oneline"))
-               return CMIT_FMT_ONELINE;
-       die("invalid --pretty format");
+       if (*arg == '=')
+               arg++;
+       for (i = 0; i < ARRAY_SIZE(cmt_fmts); i++) {
+               if (!strncmp(arg, cmt_fmts[i].n, cmt_fmts[i].cmp_len))
+                       return cmt_fmts[i].v;
+       }
+       die("invalid --pretty format: %s", arg);
  }
  
  static struct commit *check_commit(struct object *obj,
@@@ -430,10 -438,6 +439,10 @@@ static int add_user_info(const char *wh
        time = strtoul(date, &date, 10);
        tz = strtol(date, NULL, 10);
  
 +      if (fmt == CMIT_FMT_EMAIL) {
 +              what = "From";
 +              filler = "";
 +      }
        ret = sprintf(buf, "%s: %.*s%.*s\n", what,
                      (fmt == CMIT_FMT_FULLER) ? 4 : 0,
                      filler, namelen, line);
        case CMIT_FMT_MEDIUM:
                ret += sprintf(buf + ret, "Date:   %s\n", show_date(time, tz));
                break;
 +      case CMIT_FMT_EMAIL:
 +              ret += sprintf(buf + ret, "Date: %s\n",
 +                             show_rfc2822_date(time, tz));
 +              break;
        case CMIT_FMT_FULLER:
                ret += sprintf(buf + ret, "%sDate: %s\n", what, show_date(time, tz));
                break;
        return ret;
  }
  
 -static int is_empty_line(const char *line, int len)
 +static int is_empty_line(const char *line, int *len_p)
  {
 +      int len = *len_p;
        while (len && isspace(line[len-1]))
                len--;
 +      *len_p = len;
        return !len;
  }
  
@@@ -469,8 -467,7 +478,8 @@@ static int add_merge_info(enum cmit_fm
        struct commit_list *parent = commit->parents;
        int offset;
  
 -      if ((fmt == CMIT_FMT_ONELINE) || !parent || !parent->next)
 +      if ((fmt == CMIT_FMT_ONELINE) || (fmt == CMIT_FMT_EMAIL) ||
 +          !parent || !parent->next)
                return 0;
  
        offset = sprintf(buf, "Merge:");
        return offset;
  }
  
 -unsigned long pretty_print_commit(enum cmit_fmt fmt, const struct commit *commit, unsigned long len, char *buf, unsigned long space, int abbrev)
 +unsigned long pretty_print_commit(enum cmit_fmt fmt, const struct commit *commit, unsigned long len, char *buf, unsigned long space, int abbrev, const char *subject)
  {
        int hdr = 1, body = 0;
        unsigned long offset = 0;
 -      int indent = (fmt == CMIT_FMT_ONELINE) ? 0 : 4;
 +      int indent = 4;
        int parents_shown = 0;
        const char *msg = commit->buffer;
  
 +      if (fmt == CMIT_FMT_ONELINE || fmt == CMIT_FMT_EMAIL)
 +              indent = 0;
 +
        for (;;) {
                const char *line = msg;
                int linelen = get_one_line(msg, len);
                if (hdr) {
                        if (linelen == 1) {
                                hdr = 0;
 -                              if (fmt != CMIT_FMT_ONELINE)
 +                              if ((fmt != CMIT_FMT_ONELINE) && !subject)
                                        buf[offset++] = '\n';
                                continue;
                        }
                        continue;
                }
  
 -              if (is_empty_line(line, linelen)) {
 +              if (is_empty_line(line, &linelen)) {
                        if (!body)
                                continue;
 +                      if (subject)
 +                              continue;
                        if (fmt == CMIT_FMT_SHORT)
                                break;
                } else {
                        body = 1;
                }
  
 +              if (subject) {
 +                      int slen = strlen(subject);
 +                      memcpy(buf + offset, subject, slen);
 +                      offset += slen;
 +              }
                memset(buf + offset, ' ', indent);
                memcpy(buf + offset + indent, line, linelen);
                offset += linelen + indent;
 +              buf[offset++] = '\n';
                if (fmt == CMIT_FMT_ONELINE)
                        break;
 +              subject = NULL;
        }
        while (offset && isspace(buf[offset-1]))
                offset--;