TODO's and TODON'T's ...
[tig] / tig.c
diff --git a/tig.c b/tig.c
index bd0fa21..c53c04a 100644 (file)
--- a/tig.c
+++ b/tig.c
@@ -9,35 +9,34 @@
  * SYNOPSIS
  * --------
  * [verse]
- * tig
- * tig log  [git log options]
- * tig diff [git diff options]
- * tig < [git log or git diff output]
+ * tig [options]
+ * tig [options] log  [git log options]
+ * tig [options] diff [git diff options]
+ * tig [options] <    [git log or git diff output]
  *
  * DESCRIPTION
  * -----------
  * Browse changes in a git repository.
- *
- * OPTIONS
- * -------
- *
- * None.
- *
  **/
 
 #define DEBUG
-
 #ifndef DEBUG
 #define NDEBUG
 #endif
 
+#ifndef        VERSION
+#define VERSION        "tig-0.1"
+#endif
+
 #include <assert.h>
+#include <errno.h>
 #include <ctype.h>
 #include <signal.h>
 #include <stdarg.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
+#include <time.h>
 
 #include <curses.h>
 #include <form.h>
 static void die(const char *err, ...);
 static void report(const char *msg, ...);
 
-#define ARRAY_SIZE(x)  (sizeof(x) / sizeof(x[0]))
-
+/* Some ascii-shorthands that fit into the ncurses namespace. */
 #define KEY_TAB                9
 #define KEY_ESC                27
 #define KEY_DEL                127
 
-#define REQ_OFFSET     (MAX_COMMAND + 1)
+/* View requests */
+enum request {
+       /* REQ_* values from form.h is used as a basis for user actions.
+        * Offset new values below relative to MAX_COMMAND from form.h. */
+       REQ_OFFSET      = MAX_COMMAND,
+
+       /* Requests for switching between the different views. */
+       REQ_DIFF,
+       REQ_LOG,
+       REQ_MAIN,
+       REQ_VIEWS,
+
+       REQ_QUIT,
+       REQ_VERSION,
+       REQ_STOP,
+       REQ_UPDATE,
+       REQ_REDRAW,
+       REQ_FIRST_LINE,
+       REQ_LAST_LINE,
+       REQ_LINE_NUMBER,
+};
 
-/* Requests for switching between the different views. */
-#define REQ_DIFF       (REQ_OFFSET + 0)
-#define REQ_LOG                (REQ_OFFSET + 1)
-#define REQ_MAIN       (REQ_OFFSET + 2)
-#define REQ_VIEWS      (REQ_OFFSET + 3)
+/* The request are used for adressing the view array. */
+#define VIEW_OFFSET(r) ((r) - REQ_OFFSET - 1)
 
-#define REQ_QUIT       (REQ_OFFSET + 11)
-#define REQ_VERSION    (REQ_OFFSET + 12)
-#define REQ_STOP       (REQ_OFFSET + 13)
-#define REQ_UPDATE     (REQ_OFFSET + 14)
-#define REQ_REDRAW     (REQ_OFFSET + 15)
-#define REQ_FIRST_LINE (REQ_OFFSET + 16)
-#define REQ_LAST_LINE  (REQ_OFFSET + 17)
+#define SIZEOF_ID      1024
 
 #define COLOR_TRANSP   (-1)
 
+#define DATE_FORMAT    "%Y-%m-%d %H:%M"
+#define DATE_COLS      (STRING_SIZE("2006-04-29 14:21") + 1)
+
+#define ABS(x)         ((x) >= 0 ? (x) : -(x))
+#define MIN(x, y)      ((x) < (y) ? (x) : (y))
+
+#define ARRAY_SIZE(x)  (sizeof(x) / sizeof(x[0]))
+#define STRING_SIZE(x) (sizeof(x) - 1)
+
+struct commit {
+       char id[41];
+       char title[75];
+       char author[75];
+       struct tm time;
+};
+
+
+/**
+ * OPTIONS
+ * -------
+ **/
+
+static int opt_line_number;
+static int opt_request = REQ_MAIN;
+
+char head_id[SIZEOF_ID] = "HEAD";
+char commit_id[SIZEOF_ID] = "HEAD";
+
+/* Returns the index of log or diff command or -1 to exit. */
+static int
+parse_options(int argc, char *argv[])
+{
+       int i;
+
+       for (i = 1; i < argc; i++) {
+               char *opt = argv[i];
+
+               /**
+                * log [options]::
+                *      git log options.
+                **/
+               if (!strcmp(opt, "log")) {
+                       opt_request = REQ_LOG;
+                       return i;
+
+               /**
+                * diff [options]::
+                *      git diff options.
+                **/
+               } else if (!strcmp(opt, "diff")) {
+                       opt_request = REQ_DIFF;
+                       return i;
+
+               /**
+                * -l::
+                *      Start up in log view.
+                **/
+               } else if (!strcmp(opt, "-l")) {
+                       opt_request = REQ_LOG;
+
+               /**
+                * -d::
+                *      Start up in diff view.
+                **/
+               } else if (!strcmp(opt, "-d")) {
+                       opt_request = REQ_DIFF;
+
+               /**
+                * -n, --line-number::
+                *      Prefix line numbers in log and diff view.
+                **/
+               } else if (!strcmp(opt, "-n") ||
+                          !strcmp(opt, "--line-number")) {
+                       opt_line_number = 1;
+
+               /**
+                * -v, --version::
+                *      Show version and exit.
+                **/
+               } else if (!strcmp(opt, "-v") ||
+                          !strcmp(opt, "--version")) {
+                       printf("tig version %s\n", VERSION);
+                       return -1;
+
+               /**
+                * ref::
+                *      Commit reference, symbolic or raw SHA1 ID.
+                **/
+               } else if (opt[0] && opt[0] != '-') {
+                       strncpy(head_id, opt, SIZEOF_ID);
+                       strncpy(commit_id, opt, SIZEOF_ID);
+
+               } else {
+                       die("unknown command '%s'", opt);
+               }
+       }
+
+       return i;
+}
+
+
+/*
+ * Line-oriented content detection.
+ */
 
 enum line_type {
+       LINE_DEFAULT,
        LINE_AUTHOR,
+       LINE_AUTHOR_IDENT,
        LINE_COMMIT,
+       LINE_COMMITTER,
+       LINE_CURSOR,
        LINE_DATE,
        LINE_DIFF,
        LINE_DIFF_ADD,
-       LINE_DIFF_DEL,
        LINE_DIFF_CHUNK,
+       LINE_DIFF_COPY,
+       LINE_DIFF_DEL,
+       LINE_DIFF_DISSIM,
+       LINE_DIFF_NEWMODE,
+       LINE_DIFF_OLDMODE,
+       LINE_DIFF_RENAME,
+       LINE_DIFF_SIM,
        LINE_DIFF_TREE,
        LINE_INDEX,
+       LINE_MAIN_AUTHOR,
+       LINE_MAIN_COMMIT,
+       LINE_MAIN_DATE,
+       LINE_MAIN_DELIM,
+       LINE_MERGE,
        LINE_PARENT,
-       LINE_TREE,
-
-       LINE_UNKNOWN,
-       LINE_CURSOR,
+       LINE_SIGNOFF,
        LINE_STATUS,
        LINE_TITLE,
+       LINE_TREE,
 };
 
 struct line_info {
@@ -100,27 +226,47 @@ struct line_info {
 };
 
 #define LINE(type, line, fg, bg, attr) \
-       { LINE_##type, (line), sizeof(line) - 1, (fg), (bg), (attr) }
+       { LINE_##type, (line), STRING_SIZE(line), (fg), (bg), (attr) }
 
 static struct line_info line_info[] = {
-       LINE(AUTHOR,     "Author: ",    COLOR_CYAN,     COLOR_TRANSP,   0),
-       //LINE(AUTHOR,   "author ",     COLOR_CYAN,     COLOR_TRANSP,   0),
-       LINE(COMMIT,     "commit ",     COLOR_GREEN,    COLOR_TRANSP,   0),
-       LINE(DATE,       "Date:   ",    COLOR_YELLOW,   COLOR_TRANSP,   0),
-       LINE(DIFF_ADD,   "+",           COLOR_GREEN,    COLOR_TRANSP,   0),
-       LINE(DIFF_CHUNK, "@",           COLOR_MAGENTA,  COLOR_TRANSP,   0),
-       LINE(DIFF_DEL,   "-",           COLOR_RED,      COLOR_TRANSP,   0),
-       LINE(DIFF,       "diff --git ", COLOR_YELLOW,   COLOR_TRANSP,   0),
-       LINE(DIFF_TREE,  "diff-tree ",  COLOR_BLUE,     COLOR_TRANSP,   0),
-       LINE(INDEX,      "index ",      COLOR_BLUE,     COLOR_TRANSP,   0),
-       LINE(PARENT,     "parent ",     COLOR_GREEN,    COLOR_TRANSP,   0),
-       LINE(TREE,       "tree ",       COLOR_GREEN,    COLOR_TRANSP,   0),
-
-       LINE(UNKNOWN,    "",            COLOR_TRANSP,   COLOR_TRANSP,   A_NORMAL),
-
-       LINE(CURSOR,     "",            COLOR_WHITE,    COLOR_GREEN,    A_BOLD),
-       LINE(STATUS,     "",            COLOR_GREEN,    COLOR_TRANSP,   0),
-       LINE(TITLE,      "",            COLOR_YELLOW,   COLOR_BLUE,     A_BOLD),
+       /* Diff markup */
+       LINE(DIFF,         "diff --git ",       COLOR_YELLOW,   COLOR_TRANSP,   0),
+       LINE(INDEX,        "index ",            COLOR_BLUE,     COLOR_TRANSP,   0),
+       LINE(DIFF_CHUNK,   "@@",                COLOR_MAGENTA,  COLOR_TRANSP,   0),
+       LINE(DIFF_ADD,     "+",                 COLOR_GREEN,    COLOR_TRANSP,   0),
+       LINE(DIFF_DEL,     "-",                 COLOR_RED,      COLOR_TRANSP,   0),
+       LINE(DIFF_OLDMODE, "old mode ",         COLOR_YELLOW,   COLOR_TRANSP,   0),
+       LINE(DIFF_NEWMODE, "new mode ",         COLOR_YELLOW,   COLOR_TRANSP,   0),
+       LINE(DIFF_COPY,    "copy ",             COLOR_YELLOW,   COLOR_TRANSP,   0),
+       LINE(DIFF_RENAME,  "rename ",           COLOR_YELLOW,   COLOR_TRANSP,   0),
+       LINE(DIFF_SIM,     "similarity ",       COLOR_YELLOW,   COLOR_TRANSP,   0),
+       LINE(DIFF_DISSIM,  "dissimilarity ",    COLOR_YELLOW,   COLOR_TRANSP,   0),
+
+       /* Pretty print commit header */
+       LINE(AUTHOR,       "Author: ",          COLOR_CYAN,     COLOR_TRANSP,   0),
+       LINE(MERGE,        "Merge: ",           COLOR_BLUE,     COLOR_TRANSP,   0),
+       LINE(DATE,         "Date:   ",          COLOR_YELLOW,   COLOR_TRANSP,   0),
+
+       /* Raw commit header */
+       LINE(COMMIT,       "commit ",           COLOR_GREEN,    COLOR_TRANSP,   0),
+       LINE(PARENT,       "parent ",           COLOR_BLUE,     COLOR_TRANSP,   0),
+       LINE(TREE,         "tree ",             COLOR_BLUE,     COLOR_TRANSP,   0),
+       LINE(AUTHOR_IDENT, "author ",           COLOR_CYAN,     COLOR_TRANSP,   0),
+       LINE(COMMITTER,    "committer ",        COLOR_MAGENTA,  COLOR_TRANSP,   0),
+
+       /* Misc */
+       LINE(DIFF_TREE,    "diff-tree ",        COLOR_BLUE,     COLOR_TRANSP,   0),
+       LINE(SIGNOFF,      "    Signed-off-by", COLOR_YELLOW,   COLOR_TRANSP,   0),
+
+       /* UI colors */
+       LINE(DEFAULT,      "",  COLOR_TRANSP,   COLOR_TRANSP,   A_NORMAL),
+       LINE(CURSOR,       "",  COLOR_WHITE,    COLOR_GREEN,    A_BOLD),
+       LINE(STATUS,       "",  COLOR_GREEN,    COLOR_TRANSP,   0),
+       LINE(TITLE,        "",  COLOR_YELLOW,   COLOR_BLUE,     A_BOLD),
+       LINE(MAIN_DATE,    "",  COLOR_BLUE,     COLOR_TRANSP,   0),
+       LINE(MAIN_AUTHOR,  "",  COLOR_GREEN,    COLOR_TRANSP,   0),
+       LINE(MAIN_COMMIT,  "",  COLOR_TRANSP,   COLOR_TRANSP,   0),
+       LINE(MAIN_DELIM,   "",  COLOR_MAGENTA,  COLOR_TRANSP,   0),
 };
 
 static struct line_info *
@@ -131,7 +277,7 @@ get_line_info(char *line)
 
        for (i = 0; i < ARRAY_SIZE(line_info); i++) {
                if (linelen < line_info[i].linelen
-                   || strncmp(line_info[i].line, line, line_info[i].linelen))
+                   || strncasecmp(line_info[i].line, line, line_info[i].linelen))
                        continue;
 
                return &line_info[i];
@@ -145,7 +291,7 @@ get_line_type(char *line)
 {
        struct line_info *info = get_line_info(line);
 
-       return info ? info->type : LINE_UNKNOWN;
+       return info ? info->type : LINE_DEFAULT;
 }
 
 static int
@@ -184,8 +330,6 @@ init_colors(void)
 }
 
 
-
-
 /**
  * KEYS
  * ----
@@ -208,7 +352,6 @@ init_colors(void)
  *     help
  * v::
  *     version
- *
  **/
 
 #define HELP "(d)iff, (l)og, (m)ain, (q)uit, (v)ersion, (h)elp"
@@ -239,6 +382,8 @@ struct keymap keymap[] = {
        { 'l',          REQ_LOG },
        { 'm',          REQ_MAIN },
 
+       { 'n',          REQ_LINE_NUMBER },
+
        /* No input from wgetch() with nodelay() enabled. */
        { ERR,          REQ_UPDATE },
 
@@ -292,11 +437,6 @@ struct view {
        FILE *pipe;
 };
 
-struct commit {
-       char id[41];
-       char title[75];
-};
-
 static int pager_draw(struct view *view, unsigned int lineno);
 static int pager_read(struct view *view, char *line);
 
@@ -313,35 +453,28 @@ static int main_read(struct view *view, char *line);
 #define MAIN_CMD \
        "git log --stat --pretty=raw %s"
 
-/* The status window at the bottom. Used for polling keystrokes. */
+/* The status window is used for polling keystrokes. */
 static WINDOW *status_win;
-
 static WINDOW *title_win;
 
-#define SIZEOF_ID      1024
-#define SIZEOF_VIEWS   (REQ_VIEWS - REQ_OFFSET)
-
-char head_id[SIZEOF_ID] = "HEAD";
-char commit_id[SIZEOF_ID] = "HEAD";
-
-static unsigned int current_view;
+/* The number of loading views. Controls when nodelay should be in effect when
+ * polling user input. */
 static unsigned int nloading;
 
-static struct view views[];
-static struct view *display[];
-
 static struct view views[] = {
        { "diff",  DIFF_CMD,   commit_id,  pager_read,  pager_draw, sizeof(char) },
        { "log",   LOG_CMD,    head_id,    pager_read,  pager_draw, sizeof(char) },
        { "main",  MAIN_CMD,   head_id,    main_read,   main_draw,  sizeof(struct commit) },
 };
 
+/* The display array of active views and the index of the current view. */
 static struct view *display[ARRAY_SIZE(views)];
-
+static unsigned int current_view;
 
 #define foreach_view(view, i) \
        for (i = 0; i < sizeof(display) && (view = display[i]); i++)
 
+
 static void
 redraw_view(struct view *view)
 {
@@ -359,16 +492,38 @@ redraw_view(struct view *view)
        wrefresh(view->win);
 }
 
+static void
+resize_view(struct view *view)
+{
+       int lines, cols;
+
+       getmaxyx(stdscr, lines, cols);
+
+       if (view->win) {
+               mvwin(view->win, 0, 0);
+               wresize(view->win, lines - 2, cols);
+
+       } else {
+               view->win = newwin(lines - 2, 0, 0, 0);
+               if (!view->win) {
+                       report("failed to create %s view", view->name);
+                       return;
+               }
+               scrollok(view->win, TRUE);
+       }
+
+       getmaxyx(view->win, view->height, view->width);
+}
+
 /* FIXME: Fix percentage. */
 static void
 report_position(struct view *view, int all)
 {
-       report(all ? "line %d of %d (%d%%) viewing from %d"
+       report(all ? "line %d of %d (%d%%)"
                     : "line %d of %d",
               view->lineno + 1,
               view->lines,
-              view->lines ? view->offset * 100 / view->lines : 0,
-              view->offset);
+              view->lines ? (view->lineno + 1) * 100 / view->lines : 0);
 }
 
 
@@ -381,7 +536,10 @@ move_view(struct view *view, int lines)
        assert(0 <= view->offset && view->offset < view->lines);
        assert(lines);
 
-       {
+       if (view->height < ABS(lines)) {
+               redraw_view(view);
+
+       } else {
                int line = lines > 0 ? view->height - lines : 0;
                int end = line + (lines > 0 ? lines : -lines);
 
@@ -403,7 +561,7 @@ move_view(struct view *view, int lines)
                view->draw(view, view->lineno - view->offset);
        }
 
-       assert(view->offset <= view->lineno && view->lineno <= view->lines);
+       assert(view->offset <= view->lineno && view->lineno < view->lines);
 
        redrawwin(view->win);
        wrefresh(view->win);
@@ -447,7 +605,6 @@ scroll_view(struct view *view, int request)
        move_view(view, lines);
 }
 
-
 static void
 navigate_view(struct view *view, int request)
 {
@@ -481,30 +638,43 @@ navigate_view(struct view *view, int request)
                break;
        }
 
-       if (steps < 0 && view->lineno == 0) {
+       if (steps <= 0 && view->lineno == 0) {
                report("already at first line");
                return;
 
-       } else if (steps > 0 && view->lineno + 1 >= view->lines) {
+       } else if (steps >= 0 && view->lineno + 1 == view->lines) {
                report("already at last line");
                return;
        }
 
+       /* Move the current line */
        view->lineno += steps;
-       view->draw(view, view->lineno - steps - view->offset);
+       assert(0 <= view->lineno && view->lineno < view->lines);
+
+       /* Repaint the old "current" line if we be scrolling */
+       if (ABS(steps) < view->height)
+               view->draw(view, view->lineno - steps - view->offset);
 
+       /* Check whether the view needs to be scrolled */
        if (view->lineno < view->offset ||
            view->lineno >= view->offset + view->height) {
                if (steps < 0 && -steps > view->offset) {
                        steps = -view->offset;
-               } else if (steps > 0 && steps > view->height) {
-                       steps -= view->height - 1;
+
+               } else if (steps > 0) {
+                       if (view->lineno == view->lines - 1 &&
+                           view->lines > view->height) {
+                               steps = view->lines - view->offset - 1;
+                               if (steps >= view->height)
+                                       steps -= view->height - 1;
+                       }
                }
 
                move_view(view, steps);
                return;
        }
 
+       /* Draw the current line */
        view->draw(view, view->lineno - view->offset);
 
        redrawwin(view->win);
@@ -513,28 +683,6 @@ navigate_view(struct view *view, int request)
        report_position(view, view->height);
 }
 
-static void
-resize_view(struct view *view)
-{
-       int lines, cols;
-
-       getmaxyx(stdscr, lines, cols);
-
-       if (view->win) {
-               mvwin(view->win, 0, 0);
-               wresize(view->win, lines - 2, cols);
-
-       } else {
-               view->win = newwin(lines - 2, 0, 0, 0);
-               if (!view->win) {
-                       report("failed to create %s view", view->name);
-                       return;
-               }
-               scrollok(view->win, TRUE);
-       }
-
-       getmaxyx(view->win, view->height, view->width);
-}
 
 
 static bool
@@ -611,11 +759,13 @@ update_view(struct view *view)
                        break;
        }
 
-       if (redraw)
+       if (redraw) {
+               /* FIXME: This causes flickering. Draw incrementally. */
                redraw_view(view);
+       }
 
        if (ferror(view->pipe)) {
-               report("failed to read %s", view->cmd);
+               report("failed to read %s: %s", view->cmd, strerror(errno));
                goto end;
 
        } else if (feof(view->pipe)) {
@@ -637,7 +787,7 @@ end:
 static struct view *
 switch_view(struct view *prev, int request)
 {
-       struct view *view = &views[request - REQ_OFFSET];
+       struct view *view = &views[VIEW_OFFSET(request)];
        struct view *displayed;
        int i;
 
@@ -721,6 +871,11 @@ view_driver(struct view *view, int key)
                view = switch_view(view, request);
                break;
 
+       case REQ_LINE_NUMBER:
+               opt_line_number = !opt_line_number;
+               redraw_view(view);
+               break;
+
        case REQ_REDRAW:
                redraw_view(view);
                break;
@@ -763,6 +918,7 @@ pager_draw(struct view *view, unsigned int lineno)
 {
        enum line_type type;
        char *line;
+       int linelen;
        int attr;
 
        if (view->offset + lineno >= view->lines)
@@ -779,8 +935,40 @@ pager_draw(struct view *view, unsigned int lineno)
 
        attr = get_line_attr(type);
        wattrset(view->win, attr);
-       //mvwprintw(view->win, lineno, 0, "%4d: %s", view->offset + lineno, line);
-       mvwaddstr(view->win, lineno, 0, line);
+
+       linelen = strlen(line);
+       linelen = MIN(linelen, view->width);
+
+       if (opt_line_number) {
+               wmove(view->win, lineno, 0);
+               lineno += view->offset + 1;
+               if (lineno == 1 || (lineno % 10) == 0)
+                       wprintw(view->win, "%4d: ", lineno);
+               else
+                       wprintw(view->win, "    : ", lineno);
+
+               while (line) {
+                       if (*line == '\t') {
+                               waddstr(view->win, "        ");
+                               line++;
+                       } else {
+                               char *tab = strchr(line, '\t');
+
+                               if (tab)
+                                       waddnstr(view->win, line, tab - line);
+                               else
+                                       waddstr(view->win, line);
+                               line = tab;
+                       }
+               }
+               waddstr(view->win, line);
+
+       } else {
+               /* No empty lines makes cursor drawing and clearing implicit. */
+               if (!*line)
+                       line = " ", linelen = 1;
+               mvwaddnstr(view->win, lineno, 0, line, linelen);
+       }
 
        return TRUE;
 }
@@ -799,30 +987,56 @@ pager_read(struct view *view, char *line)
 static int
 main_draw(struct view *view, unsigned int lineno)
 {
+       char buf[21];
        struct commit *commit;
        enum line_type type;
+       int cols = 0;
+       size_t timelen;
 
        if (view->offset + lineno >= view->lines)
                return FALSE;
 
        commit = view->line[view->offset + lineno];
-       if (!commit) return FALSE;
+       if (!*commit->author)
+               return FALSE;
 
        if (view->offset + lineno == view->lineno) {
                strncpy(commit_id, commit->id, SIZEOF_ID);
                type = LINE_CURSOR;
        } else {
-               type = LINE_COMMIT;
+               type = LINE_MAIN_COMMIT;
        }
 
-       mvwaddch(view->win, lineno, 0, ACS_LTEE);
+       wmove(view->win, lineno, cols);
+       wattrset(view->win, get_line_attr(LINE_MAIN_DATE));
+
+       timelen = strftime(buf, sizeof(buf), DATE_FORMAT, &commit->time);
+       waddnstr(view->win, buf, timelen);
+       waddstr(view->win, " ");
+
+       cols += DATE_COLS;
+       wmove(view->win, lineno, cols);
+       wattrset(view->win, get_line_attr(LINE_MAIN_AUTHOR));
+
+       if (strlen(commit->author) > 19) {
+               waddnstr(view->win, commit->author, 18);
+               wattrset(view->win, get_line_attr(LINE_MAIN_DELIM));
+               waddch(view->win, '~');
+       } else {
+               waddstr(view->win, commit->author);
+       }
+
+       cols += 20;
+       wattrset(view->win, A_NORMAL);
+       mvwaddch(view->win, lineno, cols, ACS_LTEE);
        wattrset(view->win, get_line_attr(type));
-       mvwaddstr(view->win, lineno, 2, commit->title);
+       mvwaddstr(view->win, lineno, cols + 2, commit->title);
        wattrset(view->win, A_NORMAL);
 
        return TRUE;
 }
 
+/* Reads git log --pretty=raw output and parses it into the commit struct. */
 static int
 main_read(struct view *view, char *line)
 {
@@ -835,11 +1049,57 @@ main_read(struct view *view, char *line)
                if (!commit)
                        return FALSE;
 
+               line += STRING_SIZE("commit ");
+
                view->line[view->lines++] = commit;
-               strncpy(commit->id, line + 7, 41);
+               strncpy(commit->id, line, sizeof(commit->id));
                break;
 
+       case LINE_AUTHOR_IDENT:
+       {
+               char *ident = line + STRING_SIZE("author ");
+               char *end = strchr(ident, '<');
+
+               if (end) {
+                       for (; end > ident && isspace(end[-1]); end--) ;
+                       *end = 0;
+               }
+
+               commit = view->line[view->lines - 1];
+               strncpy(commit->author, ident, sizeof(commit->author));
+
+               /* Parse epoch and timezone */
+               if (end) {
+                       char *secs = strchr(end + 1, '>');
+                       char *zone;
+                       time_t time;
+
+                       if (!secs || secs[1] != ' ')
+                               break;
+
+                       secs += 2;
+                       time = (time_t) atol(secs);
+                       zone = strchr(secs, ' ');
+                       if (zone && strlen(zone) == STRING_SIZE(" +0700")) {
+                               long tz;
+
+                               zone++;
+                               tz  = ('0' - zone[1]) * 60 * 60 * 10;
+                               tz += ('0' - zone[2]) * 60 * 60;
+                               tz += ('0' - zone[3]) * 60;
+                               tz += ('0' - zone[4]) * 60;
+
+                               if (zone[0] == '-')
+                                       tz = -tz;
+
+                               time -= tz;
+                       }
+                       gmtime_r(&time, &commit->time);
+               }
+               break;
+       }
        default:
+               /* Fill in the commit title */
                commit = view->line[view->lines - 1];
                if (!commit->title[0] &&
                    !strncmp(line, "    ", 4) &&
@@ -889,6 +1149,11 @@ report(const char *msg, ...)
 {
        va_list args;
 
+       werase(title_win);
+       wmove(title_win, 0, 0);
+       wprintw(title_win, "commit %s", commit_id);
+       wrefresh(title_win);
+
        va_start(args, msg);
 
        werase(status_win);
@@ -902,22 +1167,23 @@ report(const char *msg, ...)
        wrefresh(status_win);
 
        va_end(args);
-
-       werase(title_win);
-       wmove(title_win, 0, 0);
-       wprintw(title_win, "commit %s", commit_id);
-       wrefresh(title_win);
-
 }
 
 int
 main(int argc, char *argv[])
 {
-       int request = REQ_MAIN;
        int x, y;
+       int request;
+       int git_cmd;
 
        signal(SIGINT, quit);
 
+       git_cmd = parse_options(argc, argv);
+       if (git_cmd < 0)
+               return 0;
+
+       request = opt_request;
+
        initscr();      /* initialize the curses library */
        nonl();         /* tell curses not to do NL->CR/NL on output */
        cbreak();       /* take input chars one at a time, no wait for \n */
@@ -973,6 +1239,35 @@ main(int argc, char *argv[])
 }
 
 /**
+ * BUGS
+ * ----
+ * Known bugs and problems:
+ *
+ * Redrawing of the main view while loading::
+ *     If only part of a commit has been parsed not all fields will be visible
+ *     or even redrawn when the whole commit have loaded. This can be
+ *     triggered when continuously moving to the last line. Use 'r' to redraw
+ *     the whole screen.
+ *
+ * TODO
+ * ----
+ * Features that should be explored.
+ *
+ *  - Proper command line handling; ability to take the command that should be
+ *    shown. Example:
+ *
+ *     $ tig log -p
+ *
+ *  - Internal command line (exmode-inspired) which allows to specify what git
+ *    log or git diff command to run. Example:
+ *
+ *     :log -p
+ *
+ * - Proper resizing support. I am yet to figure out whether catching SIGWINCH
+ *   is preferred over using ncurses' built-in support for resizing.
+ *
+ * - Locale support.
+ *
  * COPYRIGHT
  * ---------
  * Copyright (c) Jonas Fonseca, 2006
@@ -984,6 +1279,9 @@ main(int argc, char *argv[])
  *
  * SEE ALSO
  * --------
+ * [verse]
  * link:http://www.kernel.org/pub/software/scm/git/docs/[git(7)],
  * link:http://www.kernel.org/pub/software/scm/cogito/docs/[cogito(7)]
+ * gitk(1): git repository browser written using tcl/tk,
+ * gitview(1): git repository browser written using python/gtk.
  **/