Code

IO API: refactor the run request command formatter
[tig.git] / tig.c
diff --git a/tig.c b/tig.c
index 25fb237c6b1d39a2563bcf072ebcdd73614920ce..3fce6f086685395490beacd03f7afb968c5c7670 100644 (file)
--- a/tig.c
+++ b/tig.c
@@ -68,6 +68,7 @@ static int read_properties(FILE *pipe, const char *separators, int (*read)(char
 static void set_nonblocking_input(bool loading);
 static size_t utf8_length(const char *string, int *width, size_t max_width, int *trimmed, bool reserve);
 static bool prompt_yesno(const char *prompt);
+static int load_refs(void);
 
 #define ABS(x)         ((x) >= 0  ? (x) : -(x))
 #define MIN(x, y)      ((x) < (y) ? (x) :  (y))
@@ -77,7 +78,8 @@ static bool prompt_yesno(const char *prompt);
 
 #define SIZEOF_STR     1024    /* Default string size. */
 #define SIZEOF_REF     256     /* Size of symbolic or SHA1 ID. */
-#define SIZEOF_REV     41      /* Holds a SHA-1 and an ending NUL */
+#define SIZEOF_REV     41      /* Holds a SHA-1 and an ending NUL. */
+#define SIZEOF_ARG     32      /* Default argument array size. */
 
 /* Revision graph */
 
@@ -162,7 +164,16 @@ struct ref {
        unsigned int next:1;    /* For ref lists: are there more refs? */
 };
 
-static struct ref **get_refs(char *id);
+static struct ref **get_refs(const char *id);
+
+enum format_flags {
+       FORMAT_ALL,             /* Perform replacement in all arguments. */
+       FORMAT_DASH,            /* Perform replacement up until "--". */
+       FORMAT_NONE             /* No replacement should be performed. */
+};
+
+static bool format_command(char dst[], const char *src[], enum format_flags flags);
+static bool format_argv(const char *dst[], const char *src[], enum format_flags flags);
 
 struct int_map {
        const char *name;
@@ -275,6 +286,18 @@ string_enum_compare(const char *str1, const char *str2, int len)
        return 0;
 }
 
+#define prefixcmp(str1, str2) \
+       strncmp(str1, str2, STRING_SIZE(str2))
+
+static inline int
+suffixcmp(const char *str, int slen, const char *suffix)
+{
+       size_t len = slen >= 0 ? slen : strlen(str);
+       size_t suffixlen = strlen(suffix);
+
+       return suffixlen < len ? strcmp(str + len - suffixlen, suffix) : -1;
+}
+
 /* Shell quoting
  *
  * NOTE: The following is a slightly modified copy of the git project's shell
@@ -318,6 +341,147 @@ sq_quote(char buf[SIZEOF_STR], size_t bufsize, const char *src)
        return bufsize;
 }
 
+static bool
+argv_from_string(const char *argv[SIZEOF_ARG], int *argc, char *cmd)
+{
+       int valuelen;
+
+       while (*cmd && *argc < SIZEOF_ARG && (valuelen = strcspn(cmd, " \t"))) {
+               bool advance = cmd[valuelen] != 0;
+
+               cmd[valuelen] = 0;
+               argv[(*argc)++] = chomp_string(cmd);
+               cmd += valuelen + advance;
+       }
+
+       if (*argc < SIZEOF_ARG)
+               argv[*argc] = NULL;
+       return *argc < SIZEOF_ARG;
+}
+
+
+/*
+ * Executing external commands.
+ */
+
+enum io_type {
+       IO_FD,                  /* File descriptor based IO. */
+       IO_RD,                  /* Read only fork+exec IO. */
+       IO_WR,                  /* Write only fork+exec IO. */
+};
+
+struct io {
+       enum io_type type; /* The requested type of pipe. */
+       FILE *pipe;             /* Pipe for reading or writing. */
+       int error;              /* Error status. */
+       char sh[SIZEOF_STR];    /* Shell command buffer. */
+       char *buf;              /* Read/write buffer. */
+       size_t bufalloc;        /* Allocated buffer size. */
+};
+
+static void
+reset_io(struct io *io)
+{
+       io->pipe = NULL;
+       io->buf = NULL;
+       io->bufalloc = 0;
+       io->error = 0;
+}
+
+static void
+init_io(struct io *io, enum io_type type)
+{
+       reset_io(io);
+       io->type = type;
+}
+
+static bool
+init_io_fd(struct io *io, FILE *pipe)
+{
+       init_io(io, IO_FD);
+       io->pipe = pipe;
+       return io->pipe != NULL;
+}
+
+static bool
+done_io(struct io *io)
+{
+       free(io->buf);
+       if (io->type == IO_FD)
+               fclose(io->pipe);
+       else
+               pclose(io->pipe);
+       reset_io(io);
+       return TRUE;
+}
+
+static bool
+start_io(struct io *io)
+{
+       io->pipe = popen(io->sh, io->type == IO_RD ? "r" : "w");
+       return io->pipe != NULL;
+}
+
+static bool
+run_io(struct io *io, enum io_type type, const char *cmd)
+{
+       init_io(io, type);
+       string_ncopy(io->sh, cmd, strlen(cmd));
+       return start_io(io);
+}
+
+static bool
+run_io_format(struct io *io, const char *cmd, ...)
+{
+       va_list args;
+
+       va_start(args, cmd);
+       init_io(io, IO_RD);
+
+       if (vsnprintf(io->sh, sizeof(io->sh), cmd, args) >= sizeof(io->sh))
+               io->sh[0] = 0;
+       va_end(args);
+
+       return io->sh[0] ? start_io(io) : FALSE;
+}
+
+static bool
+io_eof(struct io *io)
+{
+       return feof(io->pipe);
+}
+
+static int
+io_error(struct io *io)
+{
+       return io->error;
+}
+
+static bool
+io_strerror(struct io *io)
+{
+       return strerror(io->error);
+}
+
+static char *
+io_gets(struct io *io)
+{
+       if (!io->buf) {
+               io->buf = malloc(BUFSIZ);
+               if (!io->buf)
+                       return NULL;
+               io->bufalloc = BUFSIZ;
+       }
+
+       if (!fgets(io->buf, io->bufalloc, io->pipe)) {
+               if (ferror(io->pipe))
+                       io->error = errno;
+               return NULL;
+       }
+
+       return io->buf;
+}
+
 
 /*
  * User requests
@@ -347,6 +511,13 @@ sq_quote(char buf[SIZEOF_STR], size_t bufsize, const char *src)
        REQ_(VIEW_CLOSE,        "Close the current view"), \
        REQ_(QUIT,              "Close all views and quit"), \
        \
+       REQ_GROUP("View specific requests") \
+       REQ_(STATUS_UPDATE,     "Update file status"), \
+       REQ_(STATUS_REVERT,     "Revert file changes"), \
+       REQ_(STATUS_MERGE,      "Merge file using external tool"), \
+       REQ_(STAGE_NEXT,        "Find next chunk to stage"), \
+       REQ_(TREE_PARENT,       "Switch to parent directory in tree view"), \
+       \
        REQ_GROUP("Cursor navigation") \
        REQ_(MOVE_UP,           "Move cursor one line up"), \
        REQ_(MOVE_DOWN,         "Move cursor one line down"), \
@@ -367,22 +538,19 @@ sq_quote(char buf[SIZEOF_STR], size_t bufsize, const char *src)
        REQ_(FIND_NEXT,         "Find next search match"), \
        REQ_(FIND_PREV,         "Find previous search match"), \
        \
+       REQ_GROUP("Option manipulation") \
+       REQ_(TOGGLE_LINENO,     "Toggle line numbers"), \
+       REQ_(TOGGLE_DATE,       "Toggle date display"), \
+       REQ_(TOGGLE_AUTHOR,     "Toggle author display"), \
+       REQ_(TOGGLE_REV_GRAPH,  "Toggle revision graph visualization"), \
+       REQ_(TOGGLE_REFS,       "Toggle reference display (tags/branches)"), \
+       \
        REQ_GROUP("Misc") \
        REQ_(PROMPT,            "Bring up the prompt"), \
        REQ_(SCREEN_REDRAW,     "Redraw the screen"), \
        REQ_(SCREEN_RESIZE,     "Resize the screen"), \
        REQ_(SHOW_VERSION,      "Show version information"), \
        REQ_(STOP_LOADING,      "Stop all loading views"), \
-       REQ_(TOGGLE_LINENO,     "Toggle line numbers"), \
-       REQ_(TOGGLE_DATE,       "Toggle date display"), \
-       REQ_(TOGGLE_AUTHOR,     "Toggle author display"), \
-       REQ_(TOGGLE_REV_GRAPH,  "Toggle revision graph visualization"), \
-       REQ_(TOGGLE_REFS,       "Toggle reference display (tags/branches)"), \
-       REQ_(STATUS_UPDATE,     "Update file status"), \
-       REQ_(STATUS_CHECKOUT,   "Checkout file"), \
-       REQ_(STATUS_MERGE,      "Merge file using external tool"), \
-       REQ_(STAGE_NEXT,        "Find next chunk to stage"), \
-       REQ_(TREE_PARENT,       "Switch to parent directory in tree view"), \
        REQ_(EDIT,              "Open in editor"), \
        REQ_(NONE,              "Do nothing")
 
@@ -402,9 +570,9 @@ enum request {
 
 struct request_info {
        enum request request;
-       char *name;
+       const char *name;
        int namelen;
-       char *help;
+       const char *help;
 };
 
 static struct request_info req_info[] = {
@@ -462,8 +630,8 @@ static char opt_path[SIZEOF_STR]    = "";
 static char opt_file[SIZEOF_STR]       = "";
 static char opt_ref[SIZEOF_REF]                = "";
 static char opt_head[SIZEOF_REF]       = "";
+static char opt_head_rev[SIZEOF_REV]   = "";
 static char opt_remote[SIZEOF_REF]     = "";
-static bool opt_no_head                        = TRUE;
 static FILE *opt_pipe                  = NULL;
 static char opt_encoding[20]           = "UTF-8";
 static bool opt_utf8                   = TRUE;
@@ -474,13 +642,17 @@ static char opt_cdup[SIZEOF_STR]  = "";
 static char opt_git_dir[SIZEOF_STR]    = "";
 static signed char opt_is_inside_work_tree     = -1; /* set to TRUE or FALSE */
 static char opt_editor[SIZEOF_STR]     = "";
+static FILE *opt_tty                   = NULL;
+
+#define is_initial_commit()    (!*opt_head_rev)
+#define is_head_commit(rev)    (!strcmp((rev), "HEAD") || !strcmp(opt_head_rev, (rev)))
 
 static enum request
-parse_options(int argc, char *argv[])
+parse_options(int argc, const char *argv[])
 {
        enum request request = REQ_VIEW_MAIN;
        size_t buf_size;
-       char *subcommand;
+       const char *subcommand;
        bool seen_dashdash = FALSE;
        int i;
 
@@ -534,7 +706,7 @@ parse_options(int argc, char *argv[])
        buf_size = strlen(opt_cmd);
 
        for (i = 1 + !!subcommand; i < argc; i++) {
-               char *opt = argv[i];
+               const char *opt = argv[i];
 
                if (seen_dashdash || !strcmp(opt, "--")) {
                        seen_dashdash = TRUE;
@@ -644,7 +816,7 @@ static struct line_info line_info[] = {
 };
 
 static enum line_type
-get_line_type(char *line)
+get_line_type(const char *line)
 {
        int linelen = strlen(line);
        enum line_type type;
@@ -666,7 +838,7 @@ get_line_attr(enum line_type type)
 }
 
 static struct line_info *
-get_line_info(char *name)
+get_line_info(const char *name)
 {
        size_t namelen = strlen(name);
        enum line_type type;
@@ -720,7 +892,6 @@ struct line {
 struct keybinding {
        int alias;
        enum request request;
-       struct keybinding *next;
 };
 
 static struct keybinding default_keybindings[] = {
@@ -781,7 +952,7 @@ static struct keybinding default_keybindings[] = {
        { 'F',          REQ_TOGGLE_REFS },
        { ':',          REQ_PROMPT },
        { 'u',          REQ_STATUS_UPDATE },
-       { '!',          REQ_STATUS_CHECKOUT },
+       { '!',          REQ_STATUS_REVERT },
        { 'M',          REQ_STATUS_MERGE },
        { '@',          REQ_STAGE_NEXT },
        { ',',          REQ_TREE_PARENT },
@@ -819,21 +990,23 @@ static struct int_map keymap_table[] = {
 #define set_keymap(map, name) \
        set_from_int_map(keymap_table, ARRAY_SIZE(keymap_table), map, name, strlen(name))
 
-static struct keybinding *keybindings[ARRAY_SIZE(keymap_table)];
+struct keybinding_table {
+       struct keybinding *data;
+       size_t size;
+};
+
+static struct keybinding_table keybindings[ARRAY_SIZE(keymap_table)];
 
 static void
 add_keybinding(enum keymap keymap, enum request request, int key)
 {
-       struct keybinding *keybinding;
+       struct keybinding_table *table = &keybindings[keymap];
 
-       keybinding = calloc(1, sizeof(*keybinding));
-       if (!keybinding)
+       table->data = realloc(table->data, (table->size + 1) * sizeof(*table->data));
+       if (!table->data)
                die("Failed to allocate keybinding");
-
-       keybinding->alias = key;
-       keybinding->request = request;
-       keybinding->next = keybindings[keymap];
-       keybindings[keymap] = keybinding;
+       table->data[table->size].alias = key;
+       table->data[table->size++].request = request;
 }
 
 /* Looks for a key binding first in the given map, then in the generic map, and
@@ -841,16 +1014,15 @@ add_keybinding(enum keymap keymap, enum request request, int key)
 static enum request
 get_keybinding(enum keymap keymap, int key)
 {
-       struct keybinding *kbd;
-       int i;
+       size_t i;
 
-       for (kbd = keybindings[keymap]; kbd; kbd = kbd->next)
-               if (kbd->alias == key)
-                       return kbd->request;
+       for (i = 0; i < keybindings[keymap].size; i++)
+               if (keybindings[keymap].data[i].alias == key)
+                       return keybindings[keymap].data[i].request;
 
-       for (kbd = keybindings[KEYMAP_GENERIC]; kbd; kbd = kbd->next)
-               if (kbd->alias == key)
-                       return kbd->request;
+       for (i = 0; i < keybindings[KEYMAP_GENERIC].size; i++)
+               if (keybindings[KEYMAP_GENERIC].data[i].alias == key)
+                       return keybindings[KEYMAP_GENERIC].data[i].request;
 
        for (i = 0; i < ARRAY_SIZE(default_keybindings); i++)
                if (default_keybindings[i].alias == key)
@@ -861,7 +1033,7 @@ get_keybinding(enum keymap keymap, int key)
 
 
 struct key {
-       char *name;
+       const char *name;
        int value;
 };
 
@@ -911,11 +1083,11 @@ get_key_value(const char *name)
        return ERR;
 }
 
-static char *
+static const char *
 get_key_name(int key_value)
 {
        static char key_char[] = "'X'";
-       char *seq = NULL;
+       const char *seq = NULL;
        int key;
 
        for (key = 0; key < ARRAY_SIZE(key_table); key++)
@@ -929,10 +1101,10 @@ get_key_name(int key_value)
                seq = key_char;
        }
 
-       return seq ? seq : "'?'";
+       return seq ? seq : "(no key)";
 }
 
-static char *
+static const char *
 get_key(enum request request)
 {
        static char buf[BUFSIZ];
@@ -960,34 +1132,34 @@ get_key(enum request request)
 struct run_request {
        enum keymap keymap;
        int key;
-       char cmd[SIZEOF_STR];
+       const char *argv[SIZEOF_ARG];
 };
 
 static struct run_request *run_request;
 static size_t run_requests;
 
 static enum request
-add_run_request(enum keymap keymap, int key, int argc, char **argv)
+add_run_request(enum keymap keymap, int key, int argc, const char **argv)
 {
        struct run_request *req;
-       char cmd[SIZEOF_STR];
-       size_t bufpos;
 
-       for (bufpos = 0; argc > 0; argc--, argv++)
-               if (!string_format_from(cmd, &bufpos, "%s ", *argv))
-                       return REQ_NONE;
+       if (argc >= ARRAY_SIZE(req->argv) - 1)
+               return REQ_NONE;
 
        req = realloc(run_request, (run_requests + 1) * sizeof(*run_request));
        if (!req)
                return REQ_NONE;
 
        run_request = req;
-       req = &run_request[run_requests++];
-       string_copy(req->cmd, cmd);
+       req = &run_request[run_requests];
        req->keymap = keymap;
        req->key = key;
+       req->argv[0] = NULL;
+
+       if (!format_argv(req->argv, argv, FORMAT_NONE))
+               return REQ_NONE;
 
-       return REQ_NONE + run_requests;
+       return REQ_NONE + ++run_requests;
 }
 
 static struct run_request *
@@ -1001,20 +1173,23 @@ get_run_request(enum request request)
 static void
 add_builtin_run_requests(void)
 {
+       const char *cherry_pick[] = { "git", "cherry-pick", "%(commit)", NULL };
+       const char *gc[] = { "git", "gc", NULL };
        struct {
                enum keymap keymap;
                int key;
-               char *argv[1];
+               int argc;
+               const char **argv;
        } reqs[] = {
-               { KEYMAP_MAIN,    'C', { "git cherry-pick %(commit)" } },
-               { KEYMAP_GENERIC, 'G', { "git gc" } },
+               { KEYMAP_MAIN,    'C', ARRAY_SIZE(cherry_pick) - 1, cherry_pick },
+               { KEYMAP_GENERIC, 'G', ARRAY_SIZE(gc) - 1, gc },
        };
        int i;
 
        for (i = 0; i < ARRAY_SIZE(reqs); i++) {
                enum request req;
 
-               req = add_run_request(reqs[i].keymap, reqs[i].key, 1, reqs[i].argv);
+               req = add_run_request(reqs[i].keymap, reqs[i].key, reqs[i].argc, reqs[i].argv);
                if (req != REQ_NONE)
                        add_keybinding(reqs[i].keymap, req, reqs[i].key);
        }
@@ -1056,11 +1231,11 @@ static struct int_map attr_map[] = {
 
 static int   config_lineno;
 static bool  config_errors;
-static char *config_msg;
+static const char *config_msg;
 
 /* Wants: object fgcolor bgcolor [attr] */
 static int
-option_color_command(int argc, char *argv[])
+option_color_command(int argc, const char *argv[])
 {
        struct line_info *info;
 
@@ -1113,7 +1288,7 @@ parse_int(const char *s, int default_value, int min, int max)
 
 /* Wants: name = value */
 static int
-option_set_command(int argc, char *argv[])
+option_set_command(int argc, const char *argv[])
 {
        if (argc != 3) {
                config_msg = "Wrong number of arguments given to set command";
@@ -1171,18 +1346,17 @@ option_set_command(int argc, char *argv[])
        }
 
        if (!strcmp(argv[0], "commit-encoding")) {
-               char *arg = argv[2];
-               int delimiter = *arg;
-               int i;
+               const char *arg = argv[2];
+               int arglen = strlen(arg);
 
-               switch (delimiter) {
+               switch (arg[0]) {
                case '"':
                case '\'':
-                       for (arg++, i = 0; arg[i]; i++)
-                               if (arg[i] == delimiter) {
-                                       arg[i] = 0;
-                                       break;
-                               }
+                       if (arglen == 1 || arg[arglen - 1] != arg[0]) {
+                               config_msg = "Unmatched quotation";
+                               return ERR;
+                       }
+                       arg += 1; arglen -= 2;
                default:
                        string_ncopy(opt_encoding, arg, strlen(arg));
                        return OK;
@@ -1195,7 +1369,7 @@ option_set_command(int argc, char *argv[])
 
 /* Wants: mode request key */
 static int
-option_bind_command(int argc, char *argv[])
+option_bind_command(int argc, const char *argv[])
 {
        enum request request;
        int keymap;
@@ -1244,24 +1418,14 @@ option_bind_command(int argc, char *argv[])
 }
 
 static int
-set_option(char *opt, char *value)
+set_option(const char *opt, char *value)
 {
-       char *argv[16];
-       int valuelen;
+       const char *argv[SIZEOF_ARG];
        int argc = 0;
 
-       /* Tokenize */
-       while (argc < ARRAY_SIZE(argv) && (valuelen = strcspn(value, " \t"))) {
-               argv[argc++] = value;
-               value += valuelen;
-
-               /* Nothing more to tokenize or last available token. */
-               if (!*value || argc >= ARRAY_SIZE(argv))
-                       break;
-
-               *value++ = 0;
-               while (isspace(*value))
-                       value++;
+       if (!argv_from_string(argv, &argc, value)) {
+               config_msg = "Too many option arguments";
+               return ERR;
        }
 
        if (!strcmp(opt, "color"))
@@ -1338,9 +1502,9 @@ load_option_file(const char *path)
 static int
 load_options(void)
 {
-       char *home = getenv("HOME");
-       char *tigrc_user = getenv("TIGRC_USER");
-       char *tigrc_system = getenv("TIGRC_SYSTEM");
+       const char *home = getenv("HOME");
+       const char *tigrc_user = getenv("TIGRC_USER");
+       const char *tigrc_system = getenv("TIGRC_SYSTEM");
        char buf[SIZEOF_STR];
 
        add_builtin_run_requests();
@@ -1398,7 +1562,6 @@ struct view {
        enum keymap keymap;     /* What keymap does this view have */
        bool git_dir;           /* Whether the view requires a git directory. */
 
-       char cmd[SIZEOF_STR];   /* Command buffer */
        char ref[SIZEOF_REF];   /* Hovered commit reference */
        char vid[SIZEOF_REF];   /* View ID. Set to id member when updating. */
 
@@ -1431,7 +1594,8 @@ struct view {
        unsigned long col;      /* Column when drawing. */
 
        /* Loading */
-       FILE *pipe;
+       struct io io;
+       struct io *pipe;
        time_t start_time;
 };
 
@@ -1452,14 +1616,15 @@ struct view_ops {
        void (*select)(struct view *view, struct line *line);
 };
 
-static struct view_ops pager_ops;
-static struct view_ops main_ops;
-static struct view_ops tree_ops;
-static struct view_ops blob_ops;
 static struct view_ops blame_ops;
+static struct view_ops blob_ops;
 static struct view_ops help_ops;
-static struct view_ops status_ops;
+static struct view_ops log_ops;
+static struct view_ops main_ops;
+static struct view_ops pager_ops;
 static struct view_ops stage_ops;
+static struct view_ops status_ops;
+static struct view_ops tree_ops;
 
 #define VIEW_STR(name, cmd, env, ref, ops, map, git) \
        { name, cmd, #env, ref, ops, map, git }
@@ -1471,7 +1636,7 @@ static struct view_ops stage_ops;
 static struct view views[] = {
        VIEW_(MAIN,   "main",   &main_ops,   TRUE,  ref_head),
        VIEW_(DIFF,   "diff",   &pager_ops,  TRUE,  ref_commit),
-       VIEW_(LOG,    "log",    &pager_ops,  TRUE,  ref_head),
+       VIEW_(LOG,    "log",    &log_ops,    TRUE,  ref_head),
        VIEW_(TREE,   "tree",   &tree_ops,   TRUE,  ref_commit),
        VIEW_(BLOB,   "blob",   &blob_ops,   TRUE,  ref_blob),
        VIEW_(BLAME,  "blame",  &blame_ops,  TRUE,  ref_commit),
@@ -1633,7 +1798,7 @@ draw_graphic(struct view *view, enum line_type type, chtype graphic[], size_t si
 }
 
 static bool
-draw_field(struct view *view, enum line_type type, char *text, int len, bool trim)
+draw_field(struct view *view, enum line_type type, const char *text, int len, bool trim)
 {
        int max = MIN(view->width - view->col, len);
        int col;
@@ -2178,6 +2343,121 @@ search_view(struct view *view, enum request request)
  * Incremental updating
  */
 
+static void
+reset_view(struct view *view)
+{
+       int i;
+
+       for (i = 0; i < view->lines; i++)
+               free(view->line[i].data);
+       free(view->line);
+
+       view->line = NULL;
+       view->offset = 0;
+       view->lines  = 0;
+       view->lineno = 0;
+       view->line_size = 0;
+       view->line_alloc = 0;
+       view->vid[0] = 0;
+}
+
+static void
+free_argv(const char *argv[])
+{
+       int argc;
+
+       for (argc = 0; argv[argc]; argc++)
+               free((void *) argv[argc]);
+}
+
+static bool
+format_argv(const char *dst_argv[], const char *src_argv[], enum format_flags flags)
+{
+       char buf[SIZEOF_STR];
+       int argc;
+       bool noreplace = flags == FORMAT_NONE;
+
+       free_argv(dst_argv);
+
+       for (argc = 0; src_argv[argc]; argc++) {
+               const char *arg = src_argv[argc];
+               size_t bufpos = 0;
+
+               while (arg) {
+                       char *next = strstr(arg, "%(");
+                       int len = next - arg;
+                       const char *value;
+
+                       if (!next || noreplace) {
+                               if (flags == FORMAT_DASH && !strcmp(arg, "--"))
+                                       noreplace = TRUE;
+                               len = strlen(arg);
+                               value = "";
+
+                       } else if (!prefixcmp(next, "%(directory)")) {
+                               value = opt_path;
+
+                       } else if (!prefixcmp(next, "%(file)")) {
+                               value = opt_file;
+
+                       } else if (!prefixcmp(next, "%(ref)")) {
+                               value = *opt_ref ? opt_ref : "HEAD";
+
+                       } else if (!prefixcmp(next, "%(head)")) {
+                               value = ref_head;
+
+                       } else if (!prefixcmp(next, "%(commit)")) {
+                               value = ref_commit;
+
+                       } else if (!prefixcmp(next, "%(blob)")) {
+                               value = ref_blob;
+
+                       } else {
+                               report("Unknown replacement: `%s`", next);
+                               return FALSE;
+                       }
+
+                       if (!string_format_from(buf, &bufpos, "%.*s%s", len, arg, value))
+                               return FALSE;
+
+                       arg = next && !noreplace ? strchr(next, ')') + 1 : NULL;
+               }
+
+               dst_argv[argc] = strdup(buf);
+               if (!dst_argv[argc])
+                       break;
+       }
+
+       dst_argv[argc] = NULL;
+
+       return src_argv[argc] == NULL;
+}
+
+static bool
+format_command(char dst[], const char *src_argv[], enum format_flags flags)
+{
+       const char *dst_argv[SIZEOF_ARG * 2] = { NULL };
+       int bufsize = 0;
+       int argc;
+
+       if (!format_argv(dst_argv, src_argv, flags)) {
+               free_argv(dst_argv);
+               return FALSE;
+       }
+
+       for (argc = 0; dst_argv[argc] && bufsize < SIZEOF_STR; argc++) {
+               if (bufsize > 0)
+                       dst[bufsize++] = ' ';
+               bufsize = sq_quote(dst, bufsize, dst_argv[argc]);
+       }
+
+       if (bufsize < SIZEOF_STR)
+               dst[bufsize] = 0;
+       free_argv(dst_argv);
+
+       return src_argv[argc] == NULL && bufsize < SIZEOF_STR;
+}
+
 static void
 end_update(struct view *view, bool force)
 {
@@ -2187,26 +2467,41 @@ end_update(struct view *view, bool force)
                if (!force)
                        return;
        set_nonblocking_input(FALSE);
-       if (view->pipe == stdin)
-               fclose(view->pipe);
-       else
-               pclose(view->pipe);
+       done_io(view->pipe);
        view->pipe = NULL;
 }
 
+static void
+setup_update(struct view *view, const char *vid)
+{
+       set_nonblocking_input(TRUE);
+       reset_view(view);
+       string_copy_rev(view->vid, vid);
+       view->pipe = &view->io;
+       view->start_time = time(NULL);
+}
+
 static bool
-begin_update(struct view *view)
+begin_update(struct view *view, bool refresh)
 {
-       if (opt_cmd[0]) {
-               string_copy(view->cmd, opt_cmd);
-               opt_cmd[0] = 0;
+       if (init_io_fd(&view->io, opt_pipe)) {
+               opt_pipe = NULL;
+
+       } else if (opt_cmd[0]) {
+               if (!run_io(&view->io, IO_RD, opt_cmd))
+                       return FALSE;
                /* When running random commands, initially show the
                 * command in the title. However, it maybe later be
                 * overwritten if a commit line is selected. */
                if (view == VIEW(REQ_VIEW_PAGER))
-                       string_copy(view->ref, view->cmd);
+                       string_copy(view->ref, opt_cmd);
                else
                        view->ref[0] = 0;
+               opt_cmd[0] = 0;
+
+       } else if (refresh) {
+               if (!start_io(&view->io))
+                       return FALSE;
 
        } else if (view == VIEW(REQ_VIEW_TREE)) {
                const char *format = view->cmd_env ? view->cmd_env : view->cmd_fmt;
@@ -2217,14 +2512,14 @@ begin_update(struct view *view)
                else if (sq_quote(path, 0, opt_path) >= sizeof(path))
                        return FALSE;
 
-               if (!string_format(view->cmd, format, view->id, path))
+               if (!run_io_format(&view->io, format, view->id, path))
                        return FALSE;
 
        } else {
                const char *format = view->cmd_env ? view->cmd_env : view->cmd_fmt;
                const char *id = view->id;
 
-               if (!string_format(view->cmd, format, id, id, id, id, id))
+               if (!run_io_format(&view->io, format, id, id, id, id, id))
                        return FALSE;
 
                /* Put the current ref_* value to the view title ref
@@ -2234,36 +2529,7 @@ begin_update(struct view *view)
                string_copy_rev(view->ref, view->id);
        }
 
-       /* Special case for the pager view. */
-       if (opt_pipe) {
-               view->pipe = opt_pipe;
-               opt_pipe = NULL;
-       } else {
-               view->pipe = popen(view->cmd, "r");
-       }
-
-       if (!view->pipe)
-               return FALSE;
-
-       set_nonblocking_input(TRUE);
-
-       view->offset = 0;
-       view->lines  = 0;
-       view->lineno = 0;
-       string_copy_rev(view->vid, view->id);
-
-       if (view->line) {
-               int i;
-
-               for (i = 0; i < view->lines; i++)
-                       if (view->line[i].data)
-                               free(view->line[i].data);
-
-               free(view->line);
-               view->line = NULL;
-       }
-
-       view->start_time = time(NULL);
+       setup_update(view, view->id);
 
        return TRUE;
 }
@@ -2302,7 +2568,6 @@ realloc_lines(struct view *view, size_t line_size)
 static bool
 update_view(struct view *view)
 {
-       char in_buffer[BUFSIZ];
        char out_buffer[BUFSIZ * 2];
        char *line;
        /* The number of lines to read. If too low it will cause too much
@@ -2322,7 +2587,7 @@ update_view(struct view *view)
        if (!realloc_lines(view, view->lines + lines))
                goto alloc_error;
 
-       while ((line = fgets(in_buffer, sizeof(in_buffer), view->pipe))) {
+       while ((line = io_gets(view->pipe))) {
                size_t linelen = strlen(line);
 
                if (linelen)
@@ -2365,8 +2630,17 @@ update_view(struct view *view)
                }
        }
 
+       if (io_error(view->pipe)) {
+               report("Failed to read: %s", io_strerror(view->pipe));
+               end_update(view, TRUE);
+
+       } else if (io_eof(view->pipe)) {
+               report("");
+               end_update(view, FALSE);
+       }
+
        if (!view_is_displayed(view))
-               goto check_pipe;
+               return TRUE;
 
        if (view == VIEW(REQ_VIEW_TREE)) {
                /* Clear the view and redraw everything since the tree sorting
@@ -2396,17 +2670,6 @@ update_view(struct view *view)
        /* Update the title _after_ the redraw so that if the redraw picks up a
         * commit reference in view->ref it'll be available here. */
        update_view_title(view);
-
-check_pipe:
-       if (ferror(view->pipe) && errno != 0) {
-               report("Failed to read: %s", strerror(errno));
-               end_update(view, TRUE);
-
-       } else if (feof(view->pipe)) {
-               report("");
-               end_update(view, FALSE);
-       }
-
        return TRUE;
 
 alloc_error:
@@ -2428,10 +2691,9 @@ add_line_data(struct view *view, void *data, enum line_type type)
 }
 
 static struct line *
-add_line_text(struct view *view, char *data, enum line_type type)
+add_line_text(struct view *view, const char *text, enum line_type type)
 {
-       if (data)
-               data = strdup(data);
+       char *data = text ? strdup(text) : NULL;
 
        return data ? add_line_data(view, data, type) : NULL;
 }
@@ -2446,7 +2708,8 @@ enum open_flags {
        OPEN_SPLIT = 1,         /* Split current view. */
        OPEN_BACKGROUNDED = 2,  /* Backgrounded. */
        OPEN_RELOAD = 4,        /* Reload view even if it is the current. */
-       OPEN_NOMAXIMIZE = 8     /* Do not maximize the current view. */
+       OPEN_NOMAXIMIZE = 8,    /* Do not maximize the current view. */
+       OPEN_REFRESH = 16,      /* Refresh view using previous command. */
 };
 
 static void
@@ -2454,8 +2717,8 @@ open_view(struct view *prev, enum request request, enum open_flags flags)
 {
        bool backgrounded = !!(flags & OPEN_BACKGROUNDED);
        bool split = !!(flags & OPEN_SPLIT);
-       bool reload = !!(flags & OPEN_RELOAD);
-       bool nomaximize = !!(flags & OPEN_NOMAXIMIZE);
+       bool reload = !!(flags & (OPEN_RELOAD | OPEN_REFRESH));
+       bool nomaximize = !!(flags & (OPEN_NOMAXIMIZE | OPEN_REFRESH));
        struct view *view = VIEW(request);
        int nviews = displayed_views();
        struct view *base_view = display[0];
@@ -2497,7 +2760,7 @@ open_view(struct view *prev, enum request request, enum open_flags flags)
                }
 
        } else if ((reload || strcmp(view->vid, view->id)) &&
-                  !begin_update(view)) {
+                  !begin_update(view, flags & OPEN_REFRESH)) {
                report("Failed to load %s view", view->name);
                return;
        }
@@ -2525,7 +2788,7 @@ open_view(struct view *prev, enum request request, enum open_flags flags)
                 * the screen. */
                werase(view->win);
                report("");
-       } else {
+       } else if (view_is_displayed(view)) {
                redraw_view(view);
                report("");
        }
@@ -2554,7 +2817,7 @@ open_external_viewer(const char *cmd)
        endwin();                  /* restore original tty modes */
        system(cmd);
        fprintf(stderr, "Press Enter to continue");
-       getc(stdin);
+       getc(opt_tty);
        reset_prog_mode();
        redraw_display();
 }
@@ -2576,7 +2839,7 @@ open_editor(bool from_root, const char *file)
 {
        char cmd[SIZEOF_STR];
        char file_sq[SIZEOF_STR];
-       char *editor;
+       const char *editor;
        char *prefix = from_root ? opt_cdup : "";
 
        editor = getenv("GIT_EDITOR");
@@ -2600,49 +2863,14 @@ open_run_request(enum request request)
 {
        struct run_request *req = get_run_request(request);
        char buf[SIZEOF_STR * 2];
-       size_t bufpos;
-       char *cmd;
 
        if (!req) {
                report("Unknown run request");
                return;
        }
 
-       bufpos = 0;
-       cmd = req->cmd;
-
-       while (cmd) {
-               char *next = strstr(cmd, "%(");
-               int len = next - cmd;
-               char *value;
-
-               if (!next) {
-                       len = strlen(cmd);
-                       value = "";
-
-               } else if (!strncmp(next, "%(head)", 7)) {
-                       value = ref_head;
-
-               } else if (!strncmp(next, "%(commit)", 9)) {
-                       value = ref_commit;
-
-               } else if (!strncmp(next, "%(blob)", 7)) {
-                       value = ref_blob;
-
-               } else {
-                       report("Unknown replacement in run request: `%s`", req->cmd);
-                       return;
-               }
-
-               if (!string_format_from(buf, &bufpos, "%.*s%s", len, cmd, value))
-                       return;
-
-               if (next)
-                       next = strchr(next, ')') + 1;
-               cmd = next;
-       }
-
-       open_external_viewer(buf);
+       if (format_command(buf, req->argv, FORMAT_ALL))
+               open_external_viewer(buf);
 }
 
 /*
@@ -2664,6 +2892,7 @@ view_driver(struct view *view, enum request request)
                /* FIXME: When all views can refresh always do this. */
                if (view == VIEW(REQ_VIEW_STATUS) ||
                    view == VIEW(REQ_VIEW_MAIN) ||
+                   view == VIEW(REQ_VIEW_LOG) ||
                    view == VIEW(REQ_VIEW_STAGE))
                        request = REQ_REFRESH;
                else
@@ -2873,6 +3102,7 @@ view_driver(struct view *view, enum request request)
                        view->parent = view;
                        resize_display();
                        redraw_display();
+                       report("");
                        break;
                }
                /* Fall-through */
@@ -2880,7 +3110,6 @@ view_driver(struct view *view, enum request request)
                return FALSE;
 
        default:
-               /* An unknown key will show most commonly used commands. */
                report("Unknown key, press 'h' for help");
                return TRUE;
        }
@@ -2906,7 +3135,7 @@ pager_draw(struct view *view, struct line *line, unsigned int lineno)
 }
 
 static bool
-add_describe_ref(char *buf, size_t *bufpos, char *commit_id, const char *sep)
+add_describe_ref(char *buf, size_t *bufpos, const char *commit_id, const char *sep)
 {
        char refbuf[SIZEOF_STR];
        char *ref = NULL;
@@ -2954,8 +3183,8 @@ add_pager_refs(struct view *view, struct line *line)
 
        do {
                struct ref *ref = refs[refpos];
-               char *fmt = ref->tag    ? "%s[%s]" :
-                           ref->remote ? "%s<%s>" : "%s%s";
+               const char *fmt = ref->tag    ? "%s[%s]" :
+                                 ref->remote ? "%s<%s>" : "%s%s";
 
                if (!string_format_from(buf, &bufpos, fmt, sep, ref->name))
                        return;
@@ -3066,6 +3295,29 @@ static struct view_ops pager_ops = {
        pager_select,
 };
 
+static enum request
+log_request(struct view *view, enum request request, struct line *line)
+{
+       switch (request) {
+       case REQ_REFRESH:
+               load_refs();
+               open_view(view, REQ_VIEW_LOG, OPEN_REFRESH);
+               return REQ_NONE;
+       default:
+               return pager_request(view, request, line);
+       }
+}
+
+static struct view_ops log_ops = {
+       "line",
+       NULL,
+       pager_read,
+       pager_draw,
+       log_request,
+       pager_grep,
+       pager_select,
+};
+
 
 /*
  * Help backend
@@ -3094,7 +3346,7 @@ help_open(struct view *view)
        add_line_text(view, "Quick reference for tig keybindings:", LINE_DEFAULT);
 
        for (i = 0; i < ARRAY_SIZE(req_info); i++) {
-               char *key;
+               const char *key;
 
                if (req_info[i].request == REQ_NONE)
                        continue;
@@ -3122,7 +3374,10 @@ help_open(struct view *view)
 
        for (i = 0; i < run_requests; i++) {
                struct run_request *req = get_run_request(REQ_NONE + i + 1);
-               char *key;
+               const char *key;
+               char cmd[SIZEOF_STR];
+               size_t bufpos;
+               int argc;
 
                if (!req)
                        continue;
@@ -3131,9 +3386,13 @@ help_open(struct view *view)
                if (!*key)
                        key = "(no key defined)";
 
+               for (bufpos = 0, argc = 0; req->argv[argc]; argc++)
+                       if (!string_format_from(cmd, &bufpos, "%s%s",
+                                               argc ? " " : "", req->argv[argc]))
+                               return REQ_NONE;
+
                if (!string_format(buf, "    %-10s %-14s `%s`",
-                                  keymap_table[req->keymap].name,
-                                  key, req->cmd))
+                                  keymap_table[req->keymap].name, key, cmd))
                        continue;
 
                add_line_text(view, buf, LINE_DEFAULT);
@@ -3179,7 +3438,7 @@ pop_tree_stack_entry(void)
 }
 
 static void
-push_tree_stack_entry(char *name, unsigned long lineno)
+push_tree_stack_entry(const char *name, unsigned long lineno)
 {
        struct tree_stack_entry *entry = calloc(1, sizeof(*entry));
        size_t pathlen = strlen(opt_path);
@@ -3215,8 +3474,8 @@ push_tree_stack_entry(char *name, unsigned long lineno)
 #define TREE_UP_FORMAT "040000 tree %s\t.."
 
 static int
-tree_compare_entry(enum line_type type1, char *name1,
-                  enum line_type type2, char *name2)
+tree_compare_entry(enum line_type type1, const char *name1,
+                  enum line_type type2, const char *name2)
 {
        if (type1 != type2) {
                if (type1 == LINE_TREE_DIR)
@@ -3227,10 +3486,10 @@ tree_compare_entry(enum line_type type1, char *name1,
        return strcmp(name1, name2);
 }
 
-static char *
+static const char *
 tree_path(struct line *line)
 {
-       char *path = line->data;
+       const char *path = line->data;
 
        return path + SIZEOF_TREE_ATTR;
 }
@@ -3282,7 +3541,7 @@ tree_read(struct view *view, char *text)
        /* Skip "Directory ..." and ".." line. */
        for (pos = 1 + !!*opt_path; pos < view->lines; pos++) {
                struct line *line = &view->line[pos];
-               char *path1 = tree_path(line);
+               const char *path1 = tree_path(line);
                char *path2 = text + SIZEOF_TREE_ATTR;
                int cmp = tree_compare_entry(line->type, path1, type, path2);
 
@@ -3320,30 +3579,41 @@ tree_request(struct view *view, enum request request, struct line *line)
 {
        enum open_flags flags;
 
-       if (request == REQ_VIEW_BLAME) {
-               char *filename = tree_path(line);
-
-               if (line->type == LINE_TREE_DIR) {
-                       report("Cannot show blame for directory %s", opt_path);
+       switch (request) {
+       case REQ_VIEW_BLAME:
+               if (line->type != LINE_TREE_FILE) {
+                       report("Blame only supported for files");
                        return REQ_NONE;
                }
 
                string_copy(opt_ref, view->vid);
-               string_format(opt_file, "%s%s", opt_path, filename);
                return request;
-       }
-       if (request == REQ_TREE_PARENT) {
-               if (*opt_path) {
-                       /* fake 'cd  ..' */
-                       request = REQ_ENTER;
-                       line = &view->line[1];
+
+       case REQ_EDIT:
+               if (line->type != LINE_TREE_FILE) {
+                       report("Edit only supported for files");
+               } else if (!is_head_commit(view->vid)) {
+                       report("Edit only supported for files in the current work tree");
                } else {
+                       open_editor(TRUE, opt_file);
+               }
+               return REQ_NONE;
+
+       case REQ_TREE_PARENT:
+               if (!*opt_path) {
                        /* quit view if at top of tree */
                        return REQ_VIEW_CLOSE;
                }
-       }
-       if (request != REQ_ENTER)
+               /* fake 'cd  ..' */
+               line = &view->line[1];
+               break;
+
+       case REQ_ENTER:
+               break;
+
+       default:
                return request;
+       }
 
        /* Cleanup the stack if the tree view is at a different tree. */
        while (!*opt_path && tree_stack)
@@ -3357,7 +3627,7 @@ tree_request(struct view *view, enum request request, struct line *line)
                        pop_tree_stack_entry();
 
                } else {
-                       char *basename = tree_path(line);
+                       const char *basename = tree_path(line);
 
                        push_tree_stack_entry(basename, view->lineno);
                }
@@ -3392,6 +3662,7 @@ tree_select(struct view *view, struct line *line)
 
        if (line->type == LINE_TREE_FILE) {
                string_copy_rev(ref_blob, text);
+               string_format(opt_file, "%s%s", opt_path, tree_path(line));
 
        } else if (line->type != LINE_TREE_DIR) {
                return;
@@ -3449,12 +3720,11 @@ struct blame_commit {
 
 struct blame {
        struct blame_commit *commit;
-       unsigned int header:1;
        char text[1];
 };
 
 #define BLAME_CAT_FILE_CMD "git cat-file blob %s:%s"
-#define BLAME_INCREMENTAL_CMD "git blame --incremental %s %s"
+#define BLAME_INCREMENTAL_CMD "git blame --incremental %s -- %s"
 
 static bool
 blame_open(struct view *view)
@@ -3468,40 +3738,15 @@ blame_open(struct view *view)
        if (*opt_ref && sq_quote(ref, 0, opt_ref) >= sizeof(ref))
                return FALSE;
 
-       if (*opt_ref) {
-               if (!string_format(view->cmd, BLAME_CAT_FILE_CMD, ref, path))
-                       return FALSE;
-       } else {
-               view->pipe = fopen(opt_file, "r");
-               if (!view->pipe &&
-                   !string_format(view->cmd, BLAME_CAT_FILE_CMD, "HEAD", path))
+       if (*opt_ref || !init_io_fd(&view->io, fopen(opt_file, "r"))) {
+               const char *id = *opt_ref ? ref : "HEAD";
+
+               if (!run_io_format(&view->io, BLAME_CAT_FILE_CMD, id, path))
                        return FALSE;
        }
 
-       if (!view->pipe)
-               view->pipe = popen(view->cmd, "r");
-       if (!view->pipe)
-               return FALSE;
-
-       if (!string_format(view->cmd, BLAME_INCREMENTAL_CMD, ref, path))
-               return FALSE;
-
+       setup_update(view, opt_file);
        string_format(view->ref, "%s ...", opt_file);
-       string_copy_rev(view->vid, opt_file);
-       set_nonblocking_input(TRUE);
-
-       if (view->line) {
-               int i;
-
-               for (i = 0; i < view->lines; i++)
-                       free(view->line[i].data);
-               free(view->line);
-       }
-
-       view->lines = view->line_alloc = view->line_size = view->lineno = 0;
-       view->offset = view->lines  = view->lineno = 0;
-       view->line = NULL;
-       view->start_time = time(NULL);
 
        return TRUE;
 }
@@ -3531,9 +3776,9 @@ get_blame_commit(struct view *view, const char *id)
 }
 
 static bool
-parse_number(char **posref, size_t *number, size_t min, size_t max)
+parse_number(const char **posref, size_t *number, size_t min, size_t max)
 {
-       char *pos = *posref;
+       const char *pos = *posref;
 
        *posref = NULL;
        pos = strchr(pos + 1, ' ');
@@ -3548,11 +3793,11 @@ parse_number(char **posref, size_t *number, size_t min, size_t max)
 }
 
 static struct blame_commit *
-parse_blame_commit(struct view *view, char *text, int *blamed)
+parse_blame_commit(struct view *view, const char *text, int *blamed)
 {
        struct blame_commit *commit;
        struct blame *blame;
-       char *pos = text + SIZEOF_REV - 1;
+       const char *pos = text + SIZEOF_REV - 1;
        size_t lineno;
        size_t group;
 
@@ -3573,7 +3818,6 @@ parse_blame_commit(struct view *view, char *text, int *blamed)
 
                blame = line->data;
                blame->commit = commit;
-               blame->header = !group;
                line->dirty = 1;
        }
 
@@ -3581,23 +3825,27 @@ parse_blame_commit(struct view *view, char *text, int *blamed)
 }
 
 static bool
-blame_read_file(struct view *view, char *line)
+blame_read_file(struct view *view, const char *line, bool *read_file)
 {
        if (!line) {
-               FILE *pipe = NULL;
+               char ref[SIZEOF_STR] = "";
+               char path[SIZEOF_STR];
+               struct io io = {};
 
-               if (view->lines > 0)
-                       pipe = popen(view->cmd, "r");
-               else if (!view->parent)
+               if (view->lines == 0 && !view->parent)
                        die("No blame exist for %s", view->vid);
-               view->cmd[0] = 0;
-               if (!pipe) {
+
+               if (view->lines == 0 ||
+                   sq_quote(path, 0, opt_file) >= sizeof(path) ||
+                   (*opt_ref && sq_quote(ref, 0, opt_ref) >= sizeof(ref)) ||
+                   !run_io_format(&io, BLAME_INCREMENTAL_CMD, ref, path)) {
                        report("Failed to load blame data");
                        return TRUE;
                }
 
-               fclose(view->pipe);
-               view->pipe = pipe;
+               done_io(view->pipe);
+               view->io = io;
+               *read_file = FALSE;
                return FALSE;
 
        } else {
@@ -3629,14 +3877,16 @@ blame_read(struct view *view, char *line)
        static struct blame_commit *commit = NULL;
        static int blamed = 0;
        static time_t author_time;
+       static bool read_file = TRUE;
 
-       if (*view->cmd)
-               return blame_read_file(view, line);
+       if (read_file)
+               return blame_read_file(view, line, &read_file);
 
        if (!line) {
                /* Reset all! */
                commit = NULL;
                blamed = 0;
+               read_file = TRUE;
                string_format(view->ref, "%s", view->vid);
                if (view_is_displayed(view)) {
                        update_view_title(view);
@@ -3686,7 +3936,7 @@ blame_draw(struct view *view, struct line *line, unsigned int lineno)
 {
        struct blame *blame = line->data;
        struct tm *time = NULL;
-       char *id = NULL, *author = NULL;
+       const char *id = NULL, *author = NULL;
 
        if (blame->commit && *blame->commit->filename) {
                id = blame->commit->id;
@@ -3718,12 +3968,25 @@ blame_request(struct view *view, enum request request, struct line *line)
        struct blame *blame = line->data;
 
        switch (request) {
+       case REQ_VIEW_BLAME:
+               if (!blame->commit || !strcmp(blame->commit->id, NULL_ID)) {
+                       report("Commit ID unknown");
+                       break;
+               }
+               string_copy(opt_ref, blame->commit->id);
+               open_view(view, REQ_VIEW_BLAME, OPEN_REFRESH);
+               return request;
+
        case REQ_ENTER:
                if (!blame->commit) {
                        report("No commit loaded yet");
                        break;
                }
 
+               if (view_is_displayed(VIEW(REQ_VIEW_DIFF)) &&
+                   !strcmp(blame->commit->id, VIEW(REQ_VIEW_DIFF)->ref))
+                       break;
+
                if (!strcmp(blame->commit->id, NULL_ID)) {
                        char path[SIZEOF_STR];
 
@@ -3830,13 +4093,13 @@ status_has_none(struct view *view, struct line *line)
  * :100644 100644 06a5d6ae9eca55be2e0e585a152e6b1336f2b20e 0000000000000000000000000000000000000000 M
  */
 static inline bool
-status_get_diff(struct status *file, char *buf, size_t bufsize)
+status_get_diff(struct status *file, const char *buf, size_t bufsize)
 {
-       char *old_mode = buf +  1;
-       char *new_mode = buf +  8;
-       char *old_rev  = buf + 15;
-       char *new_rev  = buf + 56;
-       char *status   = buf + 97;
+       const char *old_mode = buf +  1;
+       const char *new_mode = buf +  8;
+       const char *old_rev  = buf + 15;
+       const char *new_rev  = buf + 56;
+       const char *status   = buf + 97;
 
        if (bufsize < 99 ||
            old_mode[-1] != ':' ||
@@ -3997,19 +4260,14 @@ static bool
 status_open(struct view *view)
 {
        unsigned long prev_lineno = view->lineno;
-       size_t i;
 
-       for (i = 0; i < view->lines; i++)
-               free(view->line[i].data);
-       free(view->line);
-       view->lines = view->line_alloc = view->line_size = view->lineno = 0;
-       view->line = NULL;
+       reset_view(view);
 
        if (!realloc_lines(view, view->line_size + 7))
                return FALSE;
 
        add_line_data(view, NULL, LINE_STAT_HEAD);
-       if (opt_no_head)
+       if (is_initial_commit())
                string_copy(status_onbranch, "Initial commit");
        else if (!*opt_head)
                string_copy(status_onbranch, "Not currently on any branch");
@@ -4018,11 +4276,12 @@ status_open(struct view *view)
 
        system("git update-index -q --refresh >/dev/null 2>/dev/null");
 
-       if (opt_no_head &&
-           !status_run(view, STATUS_LIST_NO_HEAD_CMD, 'A', LINE_STAT_STAGED))
-               return FALSE;
-       else if (!status_run(view, STATUS_DIFF_INDEX_CMD, 0, LINE_STAT_STAGED))
+       if (is_initial_commit()) {
+               if (!status_run(view, STATUS_LIST_NO_HEAD_CMD, 'A', LINE_STAT_STAGED))
+                       return FALSE;
+       } else if (!status_run(view, STATUS_DIFF_INDEX_CMD, 0, LINE_STAT_STAGED)) {
                return FALSE;
+       }
 
        if (!status_run(view, STATUS_DIFF_FILES_CMD, 0, LINE_STAT_UNSTAGED) ||
            !status_run(view, STATUS_LIST_OTHER_CMD, '?', LINE_STAT_UNTRACKED))
@@ -4057,7 +4316,7 @@ status_draw(struct view *view, struct line *line, unsigned int lineno)
 {
        struct status *status = line->data;
        enum line_type type;
-       char *text;
+       const char *text;
 
        if (!status) {
                switch (line->type) {
@@ -4109,7 +4368,7 @@ status_enter(struct view *view, struct line *line)
        struct status *status = line->data;
        char oldpath[SIZEOF_STR] = "";
        char newpath[SIZEOF_STR] = "";
-       char *info;
+       const char *info;
        size_t cmdsize = 0;
        enum open_flags split;
 
@@ -4136,7 +4395,7 @@ status_enter(struct view *view, struct line *line)
 
        switch (line->type) {
        case LINE_STAT_STAGED:
-               if (opt_no_head) {
+               if (is_initial_commit()) {
                        if (!string_format_from(opt_cmd, &cmdsize,
                                                STATUS_DIFF_NO_HEAD_SHOW_CMD,
                                                newpath))
@@ -4173,6 +4432,11 @@ status_enter(struct view *view, struct line *line)
                        return REQ_NONE;
                }
 
+               if (!suffixcmp(status->new.name, -1, "/")) {
+                       report("Cannot display a directory");
+                       return REQ_NONE;
+               }
+
                opt_pipe = fopen(status->new.name, "r");
                info = "Untracked file %s";
                break;
@@ -4353,17 +4617,17 @@ status_update(struct view *view)
 }
 
 static bool
-status_checkout(struct status *status, enum line_type type, bool has_next)
+status_revert(struct status *status, enum line_type type, bool has_none)
 {
        if (!status || type != LINE_STAT_UNSTAGED) {
-               if (has_next) {
-                       report("Nothing to checkout");
+               if (type == LINE_STAT_STAGED) {
+                       report("Cannot revert changes to staged files");
                } else if (type == LINE_STAT_UNTRACKED) {
-                       report("Cannot checkout untracked files");
-               } else if (type == LINE_STAT_STAGED) {
-                       report("Cannot checkout staged files");
+                       report("Cannot revert changes to untracked files");
+               } else if (has_none) {
+                       report("Nothing to revert");
                } else {
-                       report("Cannot checkout multiple files");
+                       report("Cannot revert changes to multiple files");
                }
                return FALSE;
 
@@ -4372,7 +4636,7 @@ status_checkout(struct status *status, enum line_type type, bool has_next)
                char file_sq[SIZEOF_STR];
 
                if (sq_quote(file_sq, 0, status->old.name) >= sizeof(file_sq) ||
-                   !string_format(cmd, "git checkout %s%s", opt_cdup, file_sq))
+                   !string_format(cmd, "git checkout -- %s%s", opt_cdup, file_sq))
                        return FALSE;
 
                return run_confirm(cmd, "Are you sure you want to overwrite any changes?");
@@ -4390,8 +4654,8 @@ status_request(struct view *view, enum request request, struct line *line)
                        return REQ_NONE;
                break;
 
-       case REQ_STATUS_CHECKOUT:
-               if (!status_checkout(status, line->type, status_has_none(view, line)))
+       case REQ_STATUS_REVERT:
+               if (!status_revert(status, line->type, status_has_none(view, line)))
                        return REQ_NONE;
                break;
 
@@ -4406,6 +4670,10 @@ status_request(struct view *view, enum request request, struct line *line)
        case REQ_EDIT:
                if (!status)
                        return request;
+               if (status->status == 'D') {
+                       report("File has been deleted.");
+                       return REQ_NONE;
+               }
 
                open_editor(status->status != '?', status->new.name);
                break;
@@ -4442,8 +4710,8 @@ status_select(struct view *view, struct line *line)
 {
        struct status *status = line->data;
        char file[SIZEOF_STR] = "all files";
-       char *text;
-       char *key;
+       const char *text;
+       const char *key;
 
        if (status && !string_format(file, "'%s'", status->new.name))
                return;
@@ -4496,7 +4764,7 @@ status_grep(struct view *view, struct line *line)
                return FALSE;
 
        for (state = S_STATUS; state < S_END; state++) {
-               char *text;
+               const char *text;
 
                switch (state) {
                case S_NAME:    text = status->new.name;        break;
@@ -4530,7 +4798,7 @@ static struct view_ops status_ops = {
 static bool
 stage_diff_line(FILE *pipe, struct line *line)
 {
-       char *buf = line->data;
+       const char *buf = line->data;
        size_t bufsize = strlen(buf);
        size_t written = 0;
 
@@ -4568,7 +4836,7 @@ stage_diff_find(struct view *view, struct line *line, enum line_type type)
 }
 
 static bool
-stage_update_chunk(struct view *view, struct line *chunk)
+stage_apply_chunk(struct view *view, struct line *chunk, bool revert)
 {
        char cmd[SIZEOF_STR];
        size_t cmdsize = 0;
@@ -4584,9 +4852,10 @@ stage_update_chunk(struct view *view, struct line *chunk)
                return FALSE;
 
        if (!string_format_from(cmd, &cmdsize,
-                               "git apply --whitespace=nowarn --cached %s - && "
+                               "git apply --whitespace=nowarn %s %s - && "
                                "git update-index -q --unmerged --refresh 2>/dev/null",
-                               stage_line_type == LINE_STAT_STAGED ? "-R" : ""))
+                               revert ? "" : "--cached",
+                               revert || stage_line_type == LINE_STAT_STAGED ? "-R" : ""))
                return FALSE;
 
        pipe = popen(cmd, "w");
@@ -4607,11 +4876,11 @@ stage_update(struct view *view, struct line *line)
 {
        struct line *chunk = NULL;
 
-       if (!opt_no_head && stage_line_type != LINE_STAT_UNTRACKED)
+       if (!is_initial_commit() && stage_line_type != LINE_STAT_UNTRACKED)
                chunk = stage_diff_find(view, line, LINE_DIFF_CHUNK);
 
        if (chunk) {
-               if (!stage_update_chunk(view, chunk)) {
+               if (!stage_apply_chunk(view, chunk, FALSE)) {
                        report("Failed to apply chunk");
                        return FALSE;
                }
@@ -4636,6 +4905,31 @@ stage_update(struct view *view, struct line *line)
        return TRUE;
 }
 
+static bool
+stage_revert(struct view *view, struct line *line)
+{
+       struct line *chunk = NULL;
+
+       if (!is_initial_commit() && stage_line_type == LINE_STAT_UNSTAGED)
+               chunk = stage_diff_find(view, line, LINE_DIFF_CHUNK);
+
+       if (chunk) {
+               if (!prompt_yesno("Are you sure you want to revert changes?"))
+                       return FALSE;
+
+               if (!stage_apply_chunk(view, chunk, TRUE)) {
+                       report("Failed to revert chunk");
+                       return FALSE;
+               }
+               return TRUE;
+
+       } else {
+               return status_revert(stage_status.status ? &stage_status : NULL,
+                                    stage_line_type, FALSE);
+       }
+}
+
+
 static void
 stage_next(struct view *view, struct line *line)
 {
@@ -4681,8 +4975,8 @@ stage_request(struct view *view, enum request request, struct line *line)
                        return REQ_NONE;
                break;
 
-       case REQ_STATUS_CHECKOUT:
-               if (!status_checkout(&stage_status, stage_line_type, FALSE))
+       case REQ_STATUS_REVERT:
+               if (!stage_revert(view, line))
                        return REQ_NONE;
                break;
 
@@ -4698,6 +4992,10 @@ stage_request(struct view *view, enum request request, struct line *line)
        case REQ_EDIT:
                if (!stage_status.new.name[0])
                        return request;
+               if (stage_status.status == 'D') {
+                       report("File has been deleted.");
+                       return REQ_NONE;
+               }
 
                open_editor(stage_status.status != '?', stage_status.new.name);
                break;
@@ -4727,11 +5025,15 @@ stage_request(struct view *view, enum request request, struct line *line)
        if (!status_exists(&stage_status, stage_line_type))
                return REQ_VIEW_CLOSE;
 
-       if (stage_line_type == LINE_STAT_UNTRACKED)
+       if (stage_line_type == LINE_STAT_UNTRACKED) {
+               if (!suffixcmp(stage_status.new.name, -1, "/")) {
+                       report("Cannot display a directory");
+                       return REQ_NONE;
+               }
+
                opt_pipe = fopen(stage_status.new.name, "r");
-       else
-               string_copy(opt_cmd, view->cmd);
-       open_view(view, REQ_VIEW_STAGE, OPEN_RELOAD | OPEN_NOMAXIMIZE);
+       }
+       open_view(view, REQ_VIEW_STAGE, OPEN_REFRESH);
 
        return REQ_NONE;
 }
@@ -4829,7 +5131,7 @@ done_rev_graph(struct rev_graph *graph)
 }
 
 static void
-push_rev_graph(struct rev_graph *graph, char *parent)
+push_rev_graph(struct rev_graph *graph, const char *parent)
 {
        int i;
 
@@ -4967,8 +5269,6 @@ update_rev_graph(struct rev_graph *graph)
  * Main view backend
  */
 
-static int load_refs(void);
-
 static bool
 main_draw(struct view *view, struct line *line, unsigned int lineno)
 {
@@ -5171,8 +5471,7 @@ main_request(struct view *view, enum request request, struct line *line)
                break;
        case REQ_REFRESH:
                load_refs();
-               string_copy(opt_cmd, view->cmd);
-               open_view(view, REQ_VIEW_MAIN, OPEN_RELOAD);
+               open_view(view, REQ_VIEW_MAIN, OPEN_REFRESH);
                break;
        default:
                return request;
@@ -5489,13 +5788,13 @@ init_display(void)
        /* Initialize the curses library */
        if (isatty(STDIN_FILENO)) {
                cursed = !!initscr();
+               opt_tty = stdin;
        } else {
                /* Leave stdin and stdout alone when acting as a pager. */
-               FILE *io = fopen("/dev/tty", "r+");
-
-               if (!io)
+               opt_tty = fopen("/dev/tty", "r+");
+               if (!opt_tty)
                        die("Failed to open /dev/tty");
-               cursed = !!newterm(NULL, io, io);
+               cursed = !!newterm(NULL, opt_tty, opt_tty);
        }
 
        if (!cursed)
@@ -5656,8 +5955,27 @@ static struct ref ***id_refs = NULL;
 static size_t id_refs_alloc = 0;
 static size_t id_refs_size = 0;
 
+static int
+compare_refs(const void *ref1_, const void *ref2_)
+{
+       const struct ref *ref1 = *(const struct ref **)ref1_;
+       const struct ref *ref2 = *(const struct ref **)ref2_;
+
+       if (ref1->tag != ref2->tag)
+               return ref2->tag - ref1->tag;
+       if (ref1->ltag != ref2->ltag)
+               return ref2->ltag - ref2->ltag;
+       if (ref1->head != ref2->head)
+               return ref2->head - ref1->head;
+       if (ref1->tracked != ref2->tracked)
+               return ref2->tracked - ref1->tracked;
+       if (ref1->remote != ref2->remote)
+               return ref2->remote - ref1->remote;
+       return strcmp(ref1->name, ref2->name);
+}
+
 static struct ref **
-get_refs(char *id)
+get_refs(const char *id)
 {
        struct ref ***tmp_id_refs;
        struct ref **ref_list = NULL;
@@ -5691,19 +6009,20 @@ get_refs(char *id)
                }
 
                ref_list = tmp;
-               if (ref_list_size > 0)
-                       ref_list[ref_list_size - 1]->next = 1;
                ref_list[ref_list_size] = &refs[i];
-
                /* XXX: The properties of the commit chains ensures that we can
                 * safely modify the shared ref. The repo references will
                 * always be similar for the same id. */
-               ref_list[ref_list_size]->next = 0;
+               ref_list[ref_list_size]->next = 1;
+
                ref_list_size++;
        }
 
-       if (ref_list)
+       if (ref_list) {
+               qsort(ref_list, ref_list_size, sizeof(*ref_list), compare_refs);
+               ref_list[ref_list_size - 1]->next = 0;
                id_refs[id_refs_size++] = ref_list;
+       }
 
        return ref_list;
 }
@@ -5719,8 +6038,8 @@ read_ref(char *id, size_t idlen, char *name, size_t namelen)
        bool check_replace = FALSE;
        bool head = FALSE;
 
-       if (!strncmp(name, "refs/tags/", STRING_SIZE("refs/tags/"))) {
-               if (!strcmp(name + namelen - 3, "^{}")) {
+       if (!prefixcmp(name, "refs/tags/")) {
+               if (!suffixcmp(name, namelen, "^{}")) {
                        namelen -= 3;
                        name[namelen] = 0;
                        if (refs_size > 0 && refs[refs_size - 1].ltag == TRUE)
@@ -5733,26 +6052,26 @@ read_ref(char *id, size_t idlen, char *name, size_t namelen)
                namelen -= STRING_SIZE("refs/tags/");
                name    += STRING_SIZE("refs/tags/");
 
-       } else if (!strncmp(name, "refs/remotes/", STRING_SIZE("refs/remotes/"))) {
+       } else if (!prefixcmp(name, "refs/remotes/")) {
                remote = TRUE;
                namelen -= STRING_SIZE("refs/remotes/");
                name    += STRING_SIZE("refs/remotes/");
                tracked  = !strcmp(opt_remote, name);
 
-       } else if (!strncmp(name, "refs/heads/", STRING_SIZE("refs/heads/"))) {
+       } else if (!prefixcmp(name, "refs/heads/")) {
                namelen -= STRING_SIZE("refs/heads/");
                name    += STRING_SIZE("refs/heads/");
                head     = !strncmp(opt_head, name, namelen);
 
        } else if (!strcmp(name, "HEAD")) {
-               opt_no_head = FALSE;
+               string_ncopy(opt_head_rev, id, idlen);
                return OK;
        }
 
        if (check_replace && !strcmp(name, refs[refs_size - 1].name)) {
                /* it's an annotated tag, replace the previous sha1 with the
                 * resolved commit id; relies on the fact git-ls-remote lists
-                * the commit id of an annotated tag right beofre the commit id
+                * the commit id of an annotated tag right before the commit id
                 * it points to. */
                refs[refs_size - 1].ltag = ltag;
                string_copy_rev(refs[refs_size - 1].id, id);
@@ -5819,7 +6138,7 @@ read_repo_config_option(char *name, size_t namelen, char *value, size_t valuelen
            !strcmp(name + 7 + strlen(opt_head), ".merge")) {
                size_t from = strlen(opt_remote);
 
-               if (!strncmp(value, "refs/heads/", STRING_SIZE("refs/heads/"))) {
+               if (!prefixcmp(value, "refs/heads/")) {
                        value += STRING_SIZE("refs/heads/");
                        valuelen -= STRING_SIZE("refs/heads/");
                }
@@ -5855,7 +6174,7 @@ read_repo_info(char *name, size_t namelen, char *value, size_t valuelen)
        } else if (opt_cdup[0] == ' ') {
                string_ncopy(opt_cdup, name, namelen);
        } else {
-               if (!strncmp(name, "refs/heads/", STRING_SIZE("refs/heads/"))) {
+               if (!prefixcmp(name, "refs/heads/")) {
                        namelen -= STRING_SIZE("refs/heads/");
                        name    += STRING_SIZE("refs/heads/");
                        string_ncopy(opt_head, name, namelen);
@@ -5967,7 +6286,7 @@ warn(const char *msg, ...)
 }
 
 int
-main(int argc, char *argv[])
+main(int argc, const char *argv[])
 {
        struct view *view;
        enum request request;
@@ -6010,7 +6329,7 @@ main(int argc, char *argv[])
        if (load_refs() == ERR)
                die("Failed to load refs.");
 
-       for (i = 0; i < ARRAY_SIZE(views) && (view = &views[i]); i++)
+       foreach_view (view, i)
                view->cmd_env = getenv(view->cmd_env);
 
        init_display();
@@ -6021,6 +6340,7 @@ main(int argc, char *argv[])
 
                foreach_view (view, i)
                        update_view(view);
+               view = display[current_view];
 
                /* Refresh, accept single keystroke of input */
                key = wgetch(status_win);
@@ -6032,7 +6352,7 @@ main(int argc, char *argv[])
                        continue;
                }
 
-               request = get_keybinding(display[current_view]->keymap, key);
+               request = get_keybinding(view->keymap, key);
 
                /* Some low-level request handling. This keeps access to
                 * status_win restricted. */
@@ -6058,8 +6378,7 @@ main(int argc, char *argv[])
                case REQ_SEARCH:
                case REQ_SEARCH_BACK:
                {
-                       const char *prompt = request == REQ_SEARCH
-                                          ? "/" : "?";
+                       const char *prompt = request == REQ_SEARCH ? "/" : "?";
                        char *search = read_prompt(prompt);
 
                        if (search)