xdl_merge(): make XDL_MERGE_ZEALOUS output simpler
[gitweb.git] / builtin-shortlog.c
index 48a2a0b0d3c86f75373038bc0f18a1e9c9754ae1..fa8bc7d02a2268d99dcc829cb3adbee10c949306 100644 (file)
@@ -4,10 +4,14 @@
 #include "diff.h"
 #include "path-list.h"
 #include "revision.h"
-#include <string.h>
+#include "utf8.h"
+#include "mailmap.h"
 
 static const char shortlog_usage[] =
-"git-shortlog [-n] [-s] [<commit-id>... ]\n";
+"git-shortlog [-n] [-s] [-e] [<commit-id>... ]";
+
+static char *common_repo_prefix;
+static int email;
 
 static int compare_by_number(const void *a1, const void *a2)
 {
@@ -15,127 +19,89 @@ static int compare_by_number(const void *a1, const void *a2)
        const struct path_list *l1 = i1->util, *l2 = i2->util;
 
        if (l1->nr < l2->nr)
-               return -1;
+               return 1;
        else if (l1->nr == l2->nr)
                return 0;
        else
-               return +1;
+               return -1;
 }
 
-static struct path_list_item mailmap_list[] = {
-       { "R.Marek@sh.cvut.cz", (void*)"Rudolf Marek" },
-       { "Ralf.Wildenhues@gmx.de", (void*)"Ralf Wildenhues" },
-       { "aherrman@de.ibm.com", (void*)"Andreas Herrmann" },
-       { "akpm@osdl.org", (void*)"Andrew Morton" },
-       { "andrew.vasquez@qlogic.com", (void*)"Andrew Vasquez" },
-       { "aquynh@gmail.com", (void*)"Nguyen Anh Quynh" },
-       { "axboe@suse.de", (void*)"Jens Axboe" },
-       { "blaisorblade@yahoo.it", (void*)"Paolo 'Blaisorblade' Giarrusso" },
-       { "bunk@stusta.de", (void*)"Adrian Bunk" },
-       { "domen@coderock.org", (void*)"Domen Puncer" },
-       { "dougg@torque.net", (void*)"Douglas Gilbert" },
-       { "dwmw2@shinybook.infradead.org", (void*)"David Woodhouse" },
-       { "ecashin@coraid.com", (void*)"Ed L Cashin" },
-       { "felix@derklecks.de", (void*)"Felix Moeller" },
-       { "fzago@systemfabricworks.com", (void*)"Frank Zago" },
-       { "gregkh@suse.de", (void*)"Greg Kroah-Hartman" },
-       { "hch@lst.de", (void*)"Christoph Hellwig" },
-       { "htejun@gmail.com", (void*)"Tejun Heo" },
-       { "jejb@mulgrave.(none)", (void*)"James Bottomley" },
-       { "jejb@titanic.il.steeleye.com", (void*)"James Bottomley" },
-       { "jgarzik@pretzel.yyz.us", (void*)"Jeff Garzik" },
-       { "johnpol@2ka.mipt.ru", (void*)"Evgeniy Polyakov" },
-       { "kay.sievers@vrfy.org", (void*)"Kay Sievers" },
-       { "minyard@acm.org", (void*)"Corey Minyard" },
-       { "mshah@teja.com", (void*)"Mitesh shah" },
-       { "pj@ludd.ltu.se", (void*)"Peter A Jonsson" },
-       { "rmps@joel.ist.utl.pt", (void*)"Rui Saraiva" },
-       { "santtu.hyrkko@gmail.com", (void*)"Santtu Hyrkk\e,Av\e(B" },
-       { "simon@thekelleys.org.uk", (void*)"Simon Kelley" },
-       { "ssant@in.ibm.com", (void*)"Sachin P Sant" },
-       { "terra@gnome.org", (void*)"Morten Welinder" },
-       { "tony.luck@intel.com", (void*)"Tony Luck" },
-       { "welinder@anemone.rentec.com", (void*)"Morten Welinder" },
-       { "welinder@darter.rentec.com", (void*)"Morten Welinder" },
-       { "welinder@troll.com", (void*)"Morten Welinder" }
-};
-
-static struct path_list mailmap = {
-       mailmap_list,
-       sizeof(mailmap_list) / sizeof(struct path_list_item), 0, 0
-};
-
-static int map_email(char *email, char *name, int maxlen)
-{
-       char *p;
-       struct path_list_item *item;
-
-       /* autocomplete common developers */
-       p = strchr(email, '>');
-       if (!p)
-               return 0;
+static struct path_list mailmap = {NULL, 0, 0, 0};
 
-       *p = '\0';
-       item = path_list_lookup(email, &mailmap);
-       if (item != NULL) {
-               const char *realname = (const char *)item->util;
-               strncpy(name, realname, maxlen);
-               return 1;
-       }
-       return 0;
-}
-
-static void insert_author_oneline(struct path_list *list,
-               const char *author, int authorlen,
-               const char *oneline, int onelinelen)
+static void insert_one_record(struct path_list *list,
+                             const char *author,
+                             const char *oneline)
 {
-       const char *dot3 = "/pub/scm/linux/kernel/git/";
+       const char *dot3 = common_repo_prefix;
        char *buffer, *p;
        struct path_list_item *item;
        struct path_list *onelines;
+       char namebuf[1024];
+       size_t len;
+       const char *eol;
+       const char *boemail, *eoemail;
+
+       boemail = strchr(author, '<');
+       if (!boemail)
+               return;
+       eoemail = strchr(boemail, '>');
+       if (!eoemail)
+               return;
+       if (!map_email(&mailmap, boemail+1, namebuf, sizeof(namebuf))) {
+               while (author < boemail && isspace(*author))
+                       author++;
+               for (len = 0;
+                    len < sizeof(namebuf) - 1 && author + len < boemail;
+                    len++)
+                       namebuf[len] = author[len];
+               while (0 < len && isspace(namebuf[len-1]))
+                       len--;
+               namebuf[len] = '\0';
+       }
+       else
+               len = strlen(namebuf);
 
-       while (authorlen > 0 && isspace(author[authorlen - 1]))
-               authorlen--;
-
-       buffer = xmalloc(authorlen + 1);
-       memcpy(buffer, author, authorlen);
-       buffer[authorlen] = '\0';
+       if (email) {
+               size_t room = sizeof(namebuf) - len - 1;
+               int maillen = eoemail - boemail + 1;
+               snprintf(namebuf + len, room, " %.*s", maillen, boemail);
+       }
 
+       buffer = xstrdup(namebuf);
        item = path_list_insert(buffer, list);
        if (item->util == NULL)
                item->util = xcalloc(1, sizeof(struct path_list));
        else
                free(buffer);
 
-       if (!strncmp(oneline, "[PATCH", 6)) {
-               char *eob = strchr(buffer, ']');
-
-               while (isspace(eob[1]) && eob[1] != '\n')
-                       eob++;
-               if (eob - oneline < onelinelen) {
-                       onelinelen -= eob - oneline;
-                       oneline = eob;
-               }
-       }
-
-       while (onelinelen > 0 && isspace(oneline[0])) {
+       eol = strchr(oneline, '\n');
+       if (!eol)
+               eol = oneline + strlen(oneline);
+       while (*oneline && isspace(*oneline) && *oneline != '\n')
                oneline++;
-               onelinelen--;
+       if (!prefixcmp(oneline, "[PATCH")) {
+               char *eob = strchr(oneline, ']');
+               if (eob && (!eol || eob < eol))
+                       oneline = eob + 1;
        }
-
-       while (onelinelen > 0 && isspace(oneline[onelinelen - 1]))
-               onelinelen--;
-
-       buffer = xmalloc(onelinelen + 1);
-       memcpy(buffer, oneline, onelinelen);
-       buffer[onelinelen] = '\0';
-
-       while ((p = strstr(buffer, dot3)) != NULL) {
-               memcpy(p, "...", 3);
-               strcpy(p + 2, p + sizeof(dot3) - 1);
+       while (*oneline && isspace(*oneline) && *oneline != '\n')
+               oneline++;
+       len = eol - oneline;
+       while (len && isspace(oneline[len-1]))
+               len--;
+       buffer = xmemdupz(oneline, len);
+
+       if (dot3) {
+               int dot3len = strlen(dot3);
+               if (dot3len > 5) {
+                       while ((p = strstr(buffer, dot3)) != NULL) {
+                               int taillen = strlen(p) - dot3len;
+                               memcpy(p, "/.../", 5);
+                               memmove(p + 5, p + dot3len, taillen + 1);
+                       }
+               }
        }
 
-
        onelines = item->util;
        if (onelines->nr >= onelines->alloc) {
                onelines->alloc = alloc_nr(onelines->nr);
@@ -150,99 +116,99 @@ static void insert_author_oneline(struct path_list *list,
 
 static void read_from_stdin(struct path_list *list)
 {
-       char buffer[1024];
-
-       while (fgets(buffer, sizeof(buffer), stdin) != NULL) {
-               char *bob;
-               if ((buffer[0] == 'A' || buffer[0] == 'a') &&
-                               !strncmp(buffer + 1, "uthor: ", 7) &&
-                               (bob = strchr(buffer + 7, '<')) != NULL) {
-                       char buffer2[1024], offset = 0;
-
-                       if (map_email(bob + 1, buffer, sizeof(buffer)))
-                               bob = buffer + strlen(buffer);
-                       else {
-                               offset = 8;
-                               while (isspace(bob[-1]))
-                                       bob--;
-                       }
-
-                       while (fgets(buffer2, sizeof(buffer2), stdin) &&
-                                       buffer2[0] != '\n')
-                               ; /* chomp input */
-                       if (fgets(buffer2, sizeof(buffer2), stdin))
-                               insert_author_oneline(list,
-                                               buffer + offset,
-                                               bob - buffer - offset,
-                                               buffer2, strlen(buffer2));
-               }
+       char author[1024], oneline[1024];
+
+       while (fgets(author, sizeof(author), stdin) != NULL) {
+               if (!(author[0] == 'A' || author[0] == 'a') ||
+                   prefixcmp(author + 1, "uthor: "))
+                       continue;
+               while (fgets(oneline, sizeof(oneline), stdin) &&
+                      oneline[0] != '\n')
+                       ; /* discard headers */
+               while (fgets(oneline, sizeof(oneline), stdin) &&
+                      oneline[0] == '\n')
+                       ; /* discard blanks */
+               insert_one_record(list, author + 8, oneline);
        }
 }
 
 static void get_from_rev(struct rev_info *rev, struct path_list *list)
 {
-       char scratch[1024];
        struct commit *commit;
 
        prepare_revision_walk(rev);
        while ((commit = get_revision(rev)) != NULL) {
-               char *author = NULL, *oneline, *buffer;
-               int authorlen = authorlen, onelinelen;
+               const char *author = NULL, *buffer;
 
-               /* get author and oneline */
-               for (buffer = commit->buffer; buffer && *buffer != '\0' &&
-                               *buffer != '\n'; ) {
-                       char *eol = strchr(buffer, '\n');
+               buffer = commit->buffer;
+               while (*buffer && *buffer != '\n') {
+                       const char *eol = strchr(buffer, '\n');
 
                        if (eol == NULL)
                                eol = buffer + strlen(buffer);
                        else
                                eol++;
 
-                       if (!strncmp(buffer, "author ", 7)) {
-                               char *bracket = strchr(buffer, '<');
-
-                               if (bracket == NULL || bracket > eol)
-                                       die("Invalid commit buffer: %s",
-                                           sha1_to_hex(commit->object.sha1));
-
-                               if (map_email(bracket + 1, scratch,
-                                                       sizeof(scratch))) {
-                                       author = scratch;
-                                       authorlen = strlen(scratch);
-                               } else {
-                                       while (bracket[-1] == ' ')
-                                               bracket--;
-
-                                       author = buffer + 7;
-                                       authorlen = bracket - buffer - 7;
-                               }
-                       }
+                       if (!prefixcmp(buffer, "author "))
+                               author = buffer + 7;
                        buffer = eol;
                }
+               if (!author)
+                       die("Missing author: %s",
+                           sha1_to_hex(commit->object.sha1));
+               if (*buffer)
+                       buffer++;
+               insert_one_record(list, author, !*buffer ? "<none>" : buffer);
+       }
+}
 
-               if (author == NULL)
-                       die ("Missing author: %s",
-                                       sha1_to_hex(commit->object.sha1));
-
-               if (buffer == NULL || *buffer == '\0') {
-                       oneline = "<none>";
-                       onelinelen = sizeof(oneline) + 1;
-               } else {
-                       char *eol;
+static int parse_uint(char const **arg, int comma)
+{
+       unsigned long ul;
+       int ret;
+       char *endp;
 
-                       oneline = buffer + 1;
-                       eol = strchr(oneline, '\n');
-                       if (eol == NULL)
-                               onelinelen = strlen(oneline);
-                       else
-                               onelinelen = eol - oneline;
-               }
+       ul = strtoul(*arg, &endp, 10);
+       if (endp != *arg && *endp && *endp != comma)
+               return -1;
+       ret = (int) ul;
+       if (ret != ul)
+               return -1;
+       *arg = endp;
+       if (**arg)
+               (*arg)++;
+       return ret;
+}
 
-               insert_author_oneline(list,
-                               author, authorlen, oneline, onelinelen);
-       }
+static const char wrap_arg_usage[] = "-w[<width>[,<indent1>[,<indent2>]]]";
+#define DEFAULT_WRAPLEN 76
+#define DEFAULT_INDENT1 6
+#define DEFAULT_INDENT2 9
 
+static void parse_wrap_args(const char *arg, int *in1, int *in2, int *wrap)
+{
+       arg += 2; /* skip -w */
+
+       *wrap = parse_uint(&arg, ',');
+       if (*wrap < 0)
+               die(wrap_arg_usage);
+       *in1 = parse_uint(&arg, ',');
+       if (*in1 < 0)
+               die(wrap_arg_usage);
+       *in2 = parse_uint(&arg, '\0');
+       if (*in2 < 0)
+               die(wrap_arg_usage);
+
+       if (!*wrap)
+               *wrap = DEFAULT_WRAPLEN;
+       if (!*in1)
+               *in1 = DEFAULT_INDENT1;
+       if (!*in2)
+               *in2 = DEFAULT_INDENT2;
+       if (*wrap &&
+           ((*in1 && *wrap <= *in1) ||
+            (*in2 && *wrap <= *in2)))
+               die(wrap_arg_usage);
 }
 
 int cmd_shortlog(int argc, const char **argv, const char *prefix)
@@ -250,42 +216,71 @@ int cmd_shortlog(int argc, const char **argv, const char *prefix)
        struct rev_info rev;
        struct path_list list = { NULL, 0, 0, 1 };
        int i, j, sort_by_number = 0, summary = 0;
+       int wrap_lines = 0;
+       int wrap = DEFAULT_WRAPLEN;
+       int in1 = DEFAULT_INDENT1;
+       int in2 = DEFAULT_INDENT2;
 
-       init_revisions(&rev, prefix);
-       argc = setup_revisions(argc, argv, &rev, NULL);
+       /* since -n is a shadowed rev argument, parse our args first */
        while (argc > 1) {
                if (!strcmp(argv[1], "-n") || !strcmp(argv[1], "--numbered"))
                        sort_by_number = 1;
                else if (!strcmp(argv[1], "-s") ||
                                !strcmp(argv[1], "--summary"))
                        summary = 1;
+               else if (!strcmp(argv[1], "-e") ||
+                        !strcmp(argv[1], "--email"))
+                       email = 1;
+               else if (!prefixcmp(argv[1], "-w")) {
+                       wrap_lines = 1;
+                       parse_wrap_args(argv[1], &in1, &in2, &wrap);
+               }
                else if (!strcmp(argv[1], "-h") || !strcmp(argv[1], "--help"))
                        usage(shortlog_usage);
                else
-                       die ("unrecognized argument: %s", argv[1]);
+                       break;
                argv++;
                argc--;
        }
+       init_revisions(&rev, prefix);
+       argc = setup_revisions(argc, argv, &rev, NULL);
+       if (argc > 1)
+               die ("unrecognized argument: %s", argv[1]);
+
+       read_mailmap(&mailmap, ".mailmap", &common_repo_prefix);
 
-       if (rev.pending.nr == 1)
-               die ("Need a range!");
-       else if (rev.pending.nr == 0)
+       /* assume HEAD if from a tty */
+       if (!rev.pending.nr && isatty(0))
+               add_head_to_pending(&rev);
+       if (rev.pending.nr == 0) {
                read_from_stdin(&list);
+       }
        else
                get_from_rev(&rev, &list);
 
        if (sort_by_number)
-               qsort(list.items, sizeof(struct path_list_item), list.nr,
+               qsort(list.items, list.nr, sizeof(struct path_list_item),
                        compare_by_number);
 
        for (i = 0; i < list.nr; i++) {
                struct path_list *onelines = list.items[i].util;
 
-               printf("%s (%d):\n", list.items[i].path, onelines->nr);
-               if (!summary) {
-                       for (j = onelines->nr - 1; j >= 0; j--)
-                               printf("      %s\n", onelines->items[j].path);
-                       printf("\n");
+               if (summary) {
+                       printf("%6d\t%s\n", onelines->nr, list.items[i].path);
+               } else {
+                       printf("%s (%d):\n", list.items[i].path, onelines->nr);
+                       for (j = onelines->nr - 1; j >= 0; j--) {
+                               const char *msg = onelines->items[j].path;
+
+                               if (wrap_lines) {
+                                       int col = print_wrapped_text(msg, in1, in2, wrap);
+                                       if (col != wrap)
+                                               putchar('\n');
+                               }
+                               else
+                                       printf("      %s\n", msg);
+                       }
+                       putchar('\n');
                }
 
                onelines->strdup_paths = 1;
@@ -296,7 +291,8 @@ int cmd_shortlog(int argc, const char **argv, const char *prefix)
 
        list.strdup_paths = 1;
        path_list_clear(&list, 1);
+       mailmap.strdup_paths = 1;
+       path_list_clear(&mailmap, 1);
 
        return 0;
 }
-