Code

Read tigrc(5) options from git configuration files
[tig.git] / tig.c
diff --git a/tig.c b/tig.c
index 486b0c7444cd29526195bdb7faa61511c3505c42..83ee8b95e76dd615bdb3513356e9cb6d37a8f267 100644 (file)
--- a/tig.c
+++ b/tig.c
@@ -68,8 +68,8 @@ static void __NORETURN die(const char *err, ...);
 static void warn(const char *msg, ...);
 static void report(const char *msg, ...);
 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 int load_refs(void);
+static size_t utf8_length(const char **string, size_t col, int *width, size_t max_width, int *trimmed, bool reserve);
 
 #define ABS(x)         ((x) >= 0  ? (x) : -(x))
 #define MIN(x, y)      ((x) < (y) ? (x) :  (y))
@@ -109,6 +109,7 @@ static int load_refs(void);
 
 /* The default interval between line numbers. */
 #define NUMBER_INTERVAL        5
+#define SCROLL_INTERVAL        1
 
 #define TAB_SIZE       8
 
@@ -120,7 +121,7 @@ static int load_refs(void);
 #define GIT_CONFIG "config"
 #endif
 
-/* Some ascii-shorthands fitted into the ncurses namespace. */
+/* Some ASCII-shorthands fitted into the ncurses namespace. */
 #define KEY_TAB                '\t'
 #define KEY_RETURN     '\r'
 #define KEY_ESC                27
@@ -210,6 +211,41 @@ string_ncopy_do(char *dst, size_t dstlen, const char *src, size_t srclen)
 #define string_add(dst, from, src) \
        string_ncopy_do(dst + (from), sizeof(dst) - (from), src, sizeof(src))
 
+static size_t
+string_expand_length(const char *line, int tabsize)
+{
+       size_t size, pos;
+
+       for (pos = 0; line[pos]; pos++) {
+               if (line[pos] == '\t' && tabsize > 0)
+                       size += tabsize - (size % tabsize);
+               else
+                       size++;
+       }
+       return size;
+}
+
+static void
+string_expand(char *dst, size_t dstlen, const char *src, int tabsize)
+{
+       size_t size, pos;
+
+       for (size = pos = 0; size < dstlen - 1 && src[pos]; pos++) {
+               if (src[pos] == '\t') {
+                       size_t expanded = tabsize - (size % tabsize);
+
+                       if (expanded + size >= dstlen - 1)
+                               expanded = dstlen - size - 1;
+                       memcpy(dst + size, "        ", expanded);
+                       size += expanded;
+               } else {
+                       dst[size++] = src[pos];
+               }
+       }
+
+       dst[size] = 0;
+}
+
 static char *
 chomp_string(char *name)
 {
@@ -652,7 +688,54 @@ run_io_buf(const char **argv, char buf[], size_t bufsize)
        return done_io(&io) || error;
 }
 
-static int read_properties(struct io *io, const char *separators, int (*read)(char *, size_t, char *, size_t));
+static int
+io_load(struct io *io, const char *separators,
+       int (*read_property)(char *, size_t, char *, size_t))
+{
+       char *name;
+       int state = OK;
+
+       if (!start_io(io))
+               return ERR;
+
+       while (state == OK && (name = io_get(io, '\n', TRUE))) {
+               char *value;
+               size_t namelen;
+               size_t valuelen;
+
+               name = chomp_string(name);
+               namelen = strcspn(name, separators);
+
+               if (name[namelen]) {
+                       name[namelen] = 0;
+                       value = chomp_string(name + namelen + 1);
+                       valuelen = strlen(value);
+
+               } else {
+                       value = "";
+                       valuelen = 0;
+               }
+
+               state = read_property(name, namelen, value, valuelen);
+       }
+
+       if (state != ERR && io_error(io))
+               state = ERR;
+       done_io(io);
+
+       return state;
+}
+
+static int
+run_io_load(const char **argv, const char *separators,
+           int (*read_property)(char *, size_t, char *, size_t))
+{
+       struct io io = {};
+
+       return init_io_rd(&io, argv, NULL, FORMAT_NONE)
+               ? io_load(&io, separators, read_property) : ERR;
+}
+
 
 /*
  * User requests
@@ -698,6 +781,8 @@ static int read_properties(struct io *io, const char *separators, int (*read)(ch
        REQ_(MOVE_LAST_LINE,    "Move cursor to last line"), \
        \
        REQ_GROUP("Scrolling") \
+       REQ_(SCROLL_LEFT,       "Scroll two columns left"), \
+       REQ_(SCROLL_RIGHT,      "Scroll two columns right"), \
        REQ_(SCROLL_LINE_UP,    "Scroll one line up"), \
        REQ_(SCROLL_LINE_DOWN,  "Scroll one line down"), \
        REQ_(SCROLL_PAGE_UP,    "Scroll one page up"), \
@@ -772,19 +857,6 @@ get_request(const char *name)
  * Options
  */
 
-static const char usage[] =
-"tig " TIG_VERSION " (" __DATE__ ")\n"
-"\n"
-"Usage: tig        [options] [revs] [--] [paths]\n"
-"   or: tig show   [options] [revs] [--] [paths]\n"
-"   or: tig blame  [rev] path\n"
-"   or: tig status\n"
-"   or: tig <      [git command output]\n"
-"\n"
-"Options:\n"
-"  -v, --version   Show version and exit\n"
-"  -h, --help      Show help message and exit";
-
 /* Option and state variables. */
 static bool opt_date                   = TRUE;
 static bool opt_author                 = TRUE;
@@ -816,89 +888,6 @@ 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, const char *argv[], const char ***run_argv)
-{
-       enum request request = REQ_VIEW_MAIN;
-       const char *subcommand;
-       bool seen_dashdash = FALSE;
-       /* XXX: This is vulnerable to the user overriding options
-        * required for the main view parser. */
-       static const char *custom_argv[SIZEOF_ARG] = {
-               "git", "log", "--no-color", "--pretty=raw", "--parents",
-                       "--topo-order", NULL
-       };
-       int i, j = 6;
-
-       if (!isatty(STDIN_FILENO))
-               return REQ_VIEW_PAGER;
-
-       if (argc <= 1)
-               return REQ_VIEW_MAIN;
-
-       subcommand = argv[1];
-       if (!strcmp(subcommand, "status") || !strcmp(subcommand, "-S")) {
-               if (!strcmp(subcommand, "-S"))
-                       warn("`-S' has been deprecated; use `tig status' instead");
-               if (argc > 2)
-                       warn("ignoring arguments after `%s'", subcommand);
-               return REQ_VIEW_STATUS;
-
-       } else if (!strcmp(subcommand, "blame")) {
-               if (argc <= 2 || argc > 4)
-                       die("invalid number of options to blame\n\n%s", usage);
-
-               i = 2;
-               if (argc == 4) {
-                       string_ncopy(opt_ref, argv[i], strlen(argv[i]));
-                       i++;
-               }
-
-               string_ncopy(opt_file, argv[i], strlen(argv[i]));
-               return REQ_VIEW_BLAME;
-
-       } else if (!strcmp(subcommand, "show")) {
-               request = REQ_VIEW_DIFF;
-
-       } else if (!strcmp(subcommand, "log") || !strcmp(subcommand, "diff")) {
-               request = subcommand[0] == 'l' ? REQ_VIEW_LOG : REQ_VIEW_DIFF;
-               warn("`tig %s' has been deprecated", subcommand);
-
-       } else {
-               subcommand = NULL;
-       }
-
-       if (subcommand) {
-               custom_argv[1] = subcommand;
-               j = 2;
-       }
-
-       for (i = 1 + !!subcommand; i < argc; i++) {
-               const char *opt = argv[i];
-
-               if (seen_dashdash || !strcmp(opt, "--")) {
-                       seen_dashdash = TRUE;
-
-               } else if (!strcmp(opt, "-v") || !strcmp(opt, "--version")) {
-                       printf("tig version %s\n", TIG_VERSION);
-                       return REQ_NONE;
-
-               } else if (!strcmp(opt, "-h") || !strcmp(opt, "--help")) {
-                       printf("%s\n", usage);
-                       return REQ_NONE;
-               }
-
-               custom_argv[j++] = opt;
-               if (j >= ARRAY_SIZE(custom_argv))
-                       die("command too long");
-       }
-
-       custom_argv[j] = NULL;
-       *run_argv = custom_argv;
-
-       return request;
-}
-
 
 /*
  * Line-oriented content detection.
@@ -929,7 +918,7 @@ LINE(PP_REFS,          "Refs: ",            COLOR_RED,      COLOR_DEFAULT,  0), \
 LINE(COMMIT,      "commit ",           COLOR_GREEN,    COLOR_DEFAULT,  0), \
 LINE(PARENT,      "parent ",           COLOR_BLUE,     COLOR_DEFAULT,  0), \
 LINE(TREE,        "tree ",             COLOR_BLUE,     COLOR_DEFAULT,  0), \
-LINE(AUTHOR,      "author ",           COLOR_CYAN,     COLOR_DEFAULT,  0), \
+LINE(AUTHOR,      "author ",           COLOR_GREEN,    COLOR_DEFAULT,  0), \
 LINE(COMMITTER,           "committer ",        COLOR_MAGENTA,  COLOR_DEFAULT,  0), \
 LINE(SIGNOFF,     "    Signed-off-by", COLOR_YELLOW,   COLOR_DEFAULT,  0), \
 LINE(ACKED,       "    Acked-by",      COLOR_YELLOW,   COLOR_DEFAULT,  0), \
@@ -938,10 +927,10 @@ LINE(CURSOR,         "",                  COLOR_WHITE,    COLOR_GREEN,    A_BOLD), \
 LINE(STATUS,      "",                  COLOR_GREEN,    COLOR_DEFAULT,  0), \
 LINE(DELIMITER,           "",                  COLOR_MAGENTA,  COLOR_DEFAULT,  0), \
 LINE(DATE,         "",                 COLOR_BLUE,     COLOR_DEFAULT,  0), \
+LINE(MODE,         "",                 COLOR_CYAN,     COLOR_DEFAULT,  0), \
 LINE(LINE_NUMBER,  "",                 COLOR_CYAN,     COLOR_DEFAULT,  0), \
 LINE(TITLE_BLUR,   "",                 COLOR_WHITE,    COLOR_BLUE,     0), \
 LINE(TITLE_FOCUS,  "",                 COLOR_WHITE,    COLOR_BLUE,     A_BOLD), \
-LINE(MAIN_AUTHOR,  "",                 COLOR_GREEN,    COLOR_DEFAULT,  0), \
 LINE(MAIN_COMMIT,  "",                 COLOR_DEFAULT,  COLOR_DEFAULT,  0), \
 LINE(MAIN_TAG,     "",                 COLOR_MAGENTA,  COLOR_DEFAULT,  A_BOLD), \
 LINE(MAIN_LOCAL_TAG,"",                        COLOR_MAGENTA,  COLOR_DEFAULT,  0), \
@@ -950,8 +939,7 @@ LINE(MAIN_TRACKED, "",                      COLOR_YELLOW,   COLOR_DEFAULT,  A_BOLD), \
 LINE(MAIN_REF,     "",                 COLOR_CYAN,     COLOR_DEFAULT,  0), \
 LINE(MAIN_HEAD,    "",                 COLOR_CYAN,     COLOR_DEFAULT,  A_BOLD), \
 LINE(MAIN_REVGRAPH,"",                 COLOR_MAGENTA,  COLOR_DEFAULT,  0), \
-LINE(TREE_PARENT,  "",                 COLOR_DEFAULT,  COLOR_DEFAULT,  A_BOLD), \
-LINE(TREE_MODE,    "",                 COLOR_CYAN,     COLOR_DEFAULT,  0), \
+LINE(TREE_HEAD,    "",                 COLOR_DEFAULT,  COLOR_DEFAULT,  A_BOLD), \
 LINE(TREE_DIR,     "",                 COLOR_YELLOW,   COLOR_DEFAULT,  A_NORMAL), \
 LINE(TREE_FILE,    "",                 COLOR_DEFAULT,  COLOR_DEFAULT,  A_NORMAL), \
 LINE(STAT_HEAD,    "",                 COLOR_YELLOW,   COLOR_DEFAULT,  0), \
@@ -1100,6 +1088,8 @@ static struct keybinding default_keybindings[] = {
        { '-',          REQ_MOVE_PAGE_UP },
 
        /* Scrolling */
+       { KEY_LEFT,     REQ_SCROLL_LEFT },
+       { KEY_RIGHT,    REQ_SCROLL_RIGHT },
        { KEY_IC,       REQ_SCROLL_LINE_UP },
        { KEY_DC,       REQ_SCROLL_LINE_DOWN },
        { 'w',          REQ_SCROLL_PAGE_UP },
@@ -1401,7 +1391,7 @@ static int   config_lineno;
 static bool  config_errors;
 static const char *config_msg;
 
-/* Wants: object fgcolor bgcolor [attr] */
+/* Wants: object fgcolor bgcolor [attribute] */
 static int
 option_color_command(int argc, const char *argv[])
 {
@@ -1420,6 +1410,9 @@ option_color_command(int argc, const char *argv[])
                } else if (!string_enum_compare(argv[0], "main-date", strlen("main-date"))) {
                        info = get_line_info("date");
 
+               } else if (!string_enum_compare(argv[0], "main-author", strlen("main-author"))) {
+                       info = get_line_info("author");
+
                } else {
                        config_msg = "Unknown color name";
                        return ERR;
@@ -1518,7 +1511,7 @@ option_set_command(int argc, const char *argv[])
                int arglen = strlen(arg);
 
                switch (arg[0]) {
-               case '"':
+               case '\"':
                case '\'':
                        if (arglen == 1 || arg[arglen - 1] != arg[0]) {
                                config_msg = "Unmatched quotation";
@@ -1649,8 +1642,8 @@ read_option(char *opt, size_t optlen, char *value, size_t valuelen)
        }
 
        if (status == ERR) {
-               fprintf(stderr, "Error on line %d, near '%.*s': %s\n",
-                       config_lineno, (int) optlen, opt, config_msg);
+               warn("Error on line %d, near '%.*s': %s",
+                    config_lineno, (int) optlen, opt, config_msg);
                config_errors = TRUE;
        }
 
@@ -1663,16 +1656,16 @@ load_option_file(const char *path)
 {
        struct io io = {};
 
-       /* It's ok that the file doesn't exist. */
+       /* It's OK that the file doesn't exist. */
        if (!io_open(&io, path))
                return;
 
        config_lineno = 0;
        config_errors = FALSE;
 
-       if (read_properties(&io, " \t", read_option) == ERR ||
+       if (io_load(&io, " \t", read_option) == ERR ||
            config_errors == TRUE)
-               fprintf(stderr, "Errors while loading %s.\n", path);
+               warn("Errors while loading %s.", path);
 }
 
 static int
@@ -1743,14 +1736,16 @@ struct view {
 
        /* Navigation */
        unsigned long offset;   /* Offset of the window top */
+       unsigned long yoffset;  /* Offset from the window side. */
        unsigned long lineno;   /* Current line number */
        unsigned long p_offset; /* Previous offset of the window top */
+       unsigned long p_yoffset;/* Previous offset from the window side */
        unsigned long p_lineno; /* Previous current line number */
        bool p_restore;         /* Should the previous position be restored. */
 
        /* Searching */
        char grep[SIZEOF_STR];  /* Search string */
-       regex_t *regex;         /* Pre-compiled regex */
+       regex_t *regex;         /* Pre-compiled regexp */
 
        /* If non-NULL, points to the view that opened this view. If this view
         * is closed tig will switch back to the parent view. */
@@ -1766,6 +1761,8 @@ struct view {
        struct line *curline;   /* Line currently being drawn. */
        enum line_type curtype; /* Attribute currently used for drawing. */
        unsigned long col;      /* Column when drawing. */
+       bool has_scrolled;      /* View was scrolled. */
+       bool can_hscroll;       /* View can be scrolled horizontally. */
 
        /* Loading */
        struct io io;
@@ -1787,7 +1784,7 @@ struct view_ops {
        bool (*draw)(struct view *view, struct line *line, unsigned int lineno);
        /* Depending on view handle a special requests. */
        enum request (*request)(struct view *view, enum request request, struct line *line);
-       /* Search for regex in a line. */
+       /* Search for regexp in a line. */
        bool (*grep)(struct view *view, struct line *line);
        /* Select line */
        void (*select)(struct view *view, struct line *line);
@@ -1859,12 +1856,13 @@ draw_chars(struct view *view, enum line_type type, const char *string,
        int len = 0;
        int col = 0;
        int trimmed = FALSE;
+       size_t skip = view->yoffset > view->col ? view->yoffset - view->col : 0;
 
        if (max_len <= 0)
                return 0;
 
        if (opt_utf8) {
-               len = utf8_length(string, &col, max_len, &trimmed, use_tilde);
+               len = utf8_length(&string, skip, &col, max_len, &trimmed, use_tilde);
        } else {
                col = len = strlen(string);
                if (len > max_len) {
@@ -1877,13 +1875,17 @@ draw_chars(struct view *view, enum line_type type, const char *string,
        }
 
        set_view_attr(view, type);
-       waddnstr(view->win, string, len);
+       if (len > 0)
+               waddnstr(view->win, string, len);
        if (trimmed && use_tilde) {
                set_view_attr(view, LINE_DELIMITER);
                waddch(view->win, '~');
                col++;
        }
 
+       if (view->col + col >= view->width + view->yoffset)
+               view->can_hscroll = TRUE;
+
        return col;
 }
 
@@ -1908,6 +1910,7 @@ draw_space(struct view *view, enum line_type type, int max, int spaces)
 static bool
 draw_lineno(struct view *view, unsigned int lineno)
 {
+       size_t skip = view->yoffset > view->col ? view->yoffset - view->col : 0;
        char number[10];
        int digits3 = view->digits < 3 ? 3 : view->digits;
        int max_number = MIN(digits3, STRING_SIZE(number));
@@ -1931,29 +1934,31 @@ draw_lineno(struct view *view, unsigned int lineno)
                col = draw_space(view, LINE_LINE_NUMBER, max_number, max_number);
        }
 
-       if (col < max) {
+       if (col < max && skip <= col) {
                set_view_attr(view, LINE_DEFAULT);
                waddch(view->win, line_graphics[LINE_GRAPHIC_VLINE]);
-               col++;
        }
+       col++;
 
-       if (col < max)
-               col += draw_space(view, LINE_DEFAULT, max - col, 1);
        view->col += col;
+       if (col < max && skip <= col)
+               col = draw_space(view, LINE_DEFAULT, max - col, 1);
+       view->col++;
 
-       return view->width - view->col <= 0;
+       return view->width + view->yoffset <= view->col;
 }
 
 static bool
 draw_text(struct view *view, enum line_type type, const char *string, bool trim)
 {
-       view->col += draw_chars(view, type, string, view->width - view->col, trim);
+       view->col += draw_chars(view, type, string, view->width + view->yoffset - view->col, trim);
        return view->width - view->col <= 0;
 }
 
 static bool
 draw_graphic(struct view *view, enum line_type type, chtype graphic[], size_t size)
 {
+       size_t skip = view->yoffset > view->col ? view->yoffset - view->col : 0;
        int max = view->width - view->col;
        int i;
 
@@ -1963,14 +1968,13 @@ draw_graphic(struct view *view, enum line_type type, chtype graphic[], size_t si
        set_view_attr(view, type);
        /* Using waddch() instead of waddnstr() ensures that
         * they'll be rendered correctly for the cursor line. */
-       for (i = 0; i < size; i++)
+       for (i = skip; i < size; i++)
                waddch(view->win, graphic[i]);
 
        view->col += size;
-       if (size < max) {
+       if (size < max && skip <= size)
                waddch(view->win, ' ');
-               view->col++;
-       }
+       view->col++;
 
        return view->width - view->col <= 0;
 }
@@ -1986,8 +1990,9 @@ draw_field(struct view *view, enum line_type type, const char *text, int len, bo
        else
                col = draw_space(view, type, max - 1, max - 1);
 
-       view->col += col + draw_space(view, LINE_DEFAULT, max - col, max - col);
-       return view->width - view->col <= 0;
+       view->col += col;
+       view->col += draw_space(view, LINE_DEFAULT, max - col, max - col);
+       return view->width + view->yoffset <= view->col;
 }
 
 static bool
@@ -2027,7 +2032,28 @@ draw_author(struct view *view, const char *author)
                author = initials;
        }
 
-       return draw_field(view, LINE_MAIN_AUTHOR, author, opt_author_cols, trim);
+       return draw_field(view, LINE_AUTHOR, author, opt_author_cols, trim);
+}
+
+static bool
+draw_mode(struct view *view, mode_t mode)
+{
+       static const char dir_mode[]    = "drwxr-xr-x";
+       static const char link_mode[]   = "lrwxrwxrwx";
+       static const char exe_mode[]    = "-rwxr-xr-x";
+       static const char file_mode[]   = "-rw-r--r--";
+       const char *str;
+
+       if (S_ISDIR(mode))
+               str = dir_mode;
+       else if (S_ISLNK(mode))
+               str = link_mode;
+       else if (mode & S_IXUSR)
+               str = exe_mode;
+       else
+               str = file_mode;
+
+       return draw_field(view, LINE_MODE, str, sizeof(file_mode), FALSE);
 }
 
 static bool
@@ -2087,6 +2113,9 @@ redraw_view_from(struct view *view, int lineno)
 {
        assert(0 <= lineno && lineno < view->height);
 
+       if (lineno == 0)
+               view->can_hscroll = FALSE;
+
        for (; lineno < view->height; lineno++) {
                if (!draw_view_line(view, lineno))
                        break;
@@ -2103,19 +2132,6 @@ redraw_view(struct view *view)
 }
 
 
-static void
-update_display_cursor(struct view *view)
-{
-       /* Move the cursor to the right-most column of the cursor line.
-        *
-        * XXX: This could turn out to be a bit expensive, but it ensures that
-        * the cursor does not jump around. */
-       if (view->lines) {
-               wmove(view->win, view->lineno - view->offset, view->width - 1);
-               wnoutrefresh(view->win);
-       }
-}
-
 static void
 update_view_title(struct view *view)
 {
@@ -2168,7 +2184,6 @@ update_view_title(struct view *view)
 
        mvwaddnstr(view->title, 0, 0, buf, bufpos);
        wclrtoeol(view->title);
-       wmove(view->title, 0, view->width - 1);
        wnoutrefresh(view->title);
 }
 
@@ -2235,9 +2250,6 @@ redraw_display(bool clear)
                redraw_view(view);
                update_view_title(view);
        }
-
-       if (display[current_view])
-               update_display_cursor(display[current_view]);
 }
 
 static void
@@ -2295,6 +2307,7 @@ do_scroll_view(struct view *view, int lines)
                wnoutrefresh(view->win);
        }
 
+       view->has_scrolled = TRUE;
        report("");
 }
 
@@ -2307,6 +2320,27 @@ scroll_view(struct view *view, enum request request)
        assert(view_is_displayed(view));
 
        switch (request) {
+       case REQ_SCROLL_LEFT:
+               if (view->yoffset == 0) {
+                       report("Cannot scroll beyond the first column");
+                       return;
+               }
+               if (view->yoffset <= SCROLL_INTERVAL)
+                       view->yoffset = 0;
+               else
+                       view->yoffset -= SCROLL_INTERVAL;
+               redraw_view_from(view, 0);
+               report("");
+               return;
+       case REQ_SCROLL_RIGHT:
+               if (!view->can_hscroll) {
+                       report("Cannot scroll beyond the last column");
+                       return;
+               }
+               view->yoffset += SCROLL_INTERVAL;
+               redraw_view(view);
+               report("");
+               return;
        case REQ_SCROLL_PAGE_DOWN:
                lines = view->height;
        case REQ_SCROLL_LINE_DOWN:
@@ -2548,10 +2582,12 @@ reset_view(struct view *view)
        free(view->line);
 
        view->p_offset = view->offset;
+       view->p_yoffset = view->yoffset;
        view->p_lineno = view->lineno;
 
        view->line = NULL;
        view->offset = 0;
+       view->yoffset = 0;
        view->lines  = 0;
        view->lineno = 0;
        view->line_alloc = 0;
@@ -2662,6 +2698,7 @@ restore_view_position(struct view *view)
                werase(view->win);
 
        view->offset = view->p_offset;
+       view->yoffset = view->p_yoffset;
        view->lineno = view->p_lineno;
        view->p_restore = FALSE;
 
@@ -2786,7 +2823,7 @@ update_view(struct view *view)
                if (view->lines == 0) {
                        time_t secs = time(NULL) - view->start_time;
 
-                       if (secs > view->update_secs) {
+                       if (secs > 1 && secs > view->update_secs) {
                                if (view->update_secs == 0)
                                        redraw_view(view);
                                update_view_title(view);
@@ -2856,7 +2893,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);
-       update_display_cursor(view);
        return TRUE;
 }
 
@@ -3099,6 +3135,8 @@ view_driver(struct view *view, enum request request)
                move_view(view, request);
                break;
 
+       case REQ_SCROLL_LEFT:
+       case REQ_SCROLL_RIGHT:
        case REQ_SCROLL_LINE_DOWN:
        case REQ_SCROLL_LINE_UP:
        case REQ_SCROLL_PAGE_DOWN:
@@ -3405,11 +3443,12 @@ select_commit_parent(const char *id, char rev[SIZEOF_REV])
 static bool
 pager_draw(struct view *view, struct line *line, unsigned int lineno)
 {
-       char *text = line->data;
+       char text[SIZEOF_STR];
 
        if (opt_line_number && draw_lineno(view, lineno))
                return TRUE;
 
+       string_expand(text, sizeof(text), line->data, opt_tab_size);
        draw_text(view, line->type, text, TRUE);
        return TRUE;
 }
@@ -3742,14 +3781,11 @@ push_tree_stack_entry(const char *name, unsigned long lineno)
 
 /* Parse output from git-ls-tree(1):
  *
- * 100644 blob fb0e31ea6cc679b7379631188190e975f5789c26        Makefile
- * 100644 blob 5304ca4260aaddaee6498f9630e7d471b8591ea6        README
  * 100644 blob f931e1d229c3e185caad4449bf5b66ed72462657        tig.c
- * 100644 blob ed09fe897f3c7c9af90bcf80cae92558ea88ae38        web.conf
  */
 
 #define SIZEOF_TREE_ATTR \
-       STRING_SIZE("100644 blob ed09fe897f3c7c9af90bcf80cae92558ea88ae38\t")
+       STRING_SIZE("100644 blob f931e1d229c3e185caad4449bf5b66ed72462657\t")
 
 #define SIZEOF_TREE_MODE \
        STRING_SIZE("100644 ")
@@ -3820,6 +3856,12 @@ tree_read_date(struct view *view, char *text, bool *read_date)
                };
                struct io io = {};
 
+               if (!view->lines) {
+                       tree_entry(view, LINE_TREE_HEAD, opt_path, NULL, NULL);
+                       report("Tree is empty");
+                       return TRUE;
+               }
+
                if (!run_io_rd(&io, log_file, FORMAT_NONE)) {
                        report("Failed to load tree data");
                        return TRUE;
@@ -3887,7 +3929,7 @@ tree_read(struct view *view, char *text)
        if (textlen <= SIZEOF_TREE_ATTR)
                return FALSE;
        if (view->lines == 0 &&
-           !tree_entry(view, LINE_TREE_PARENT, opt_path, NULL, NULL))
+           !tree_entry(view, LINE_TREE_HEAD, opt_path, NULL, NULL))
                return FALSE;
 
        /* Strip the path part ... */
@@ -3938,27 +3980,11 @@ tree_draw(struct view *view, struct line *line, unsigned int lineno)
 {
        struct tree_entry *entry = line->data;
 
-       if (line->type == LINE_TREE_PARENT) {
+       if (line->type == LINE_TREE_HEAD) {
                if (draw_text(view, line->type, "Directory path /", TRUE))
                        return TRUE;
        } else {
-               char mode[11] = "-r--r--r--";
-
-               if (S_ISDIR(entry->mode)) {
-                       mode[3] = mode[6] = mode[9] = 'x';
-                       mode[0] = 'd';
-               }
-               if (S_ISLNK(entry->mode))
-                       mode[0] = 'l';
-               if (entry->mode & S_IWUSR)
-                       mode[2] = 'w';
-               if (entry->mode & S_IXUSR)
-                       mode[3] = 'x';
-               if (entry->mode & S_IXGRP)
-                       mode[6] = 'x';
-               if (entry->mode & S_IXOTH)
-                       mode[9] = 'x';
-               if (draw_field(view, LINE_TREE_MODE, mode, 11, TRUE))
+               if (draw_mode(view, entry->mode))
                        return TRUE;
 
                if (opt_author && draw_author(view, entry->author))
@@ -4035,7 +4061,7 @@ tree_request(struct view *view, enum request request, struct line *line)
 
        switch (line->type) {
        case LINE_TREE_DIR:
-               /* Depending on whether it is a subdir or parent (updir?) link
+               /* Depending on whether it is a subdirectory or parent link
                 * mangle the path buffer. */
                if (line == &view->line[1] && *opt_path) {
                        pop_tree_stack_entry();
@@ -4279,12 +4305,14 @@ blame_read_file(struct view *view, const char *line, bool *read_file)
                return FALSE;
 
        } else {
-               size_t linelen = strlen(line);
+               size_t linelen = string_expand_length(line, opt_tab_size);
                struct blame *blame = malloc(sizeof(*blame) + linelen);
 
+               if (!blame)
+                       return FALSE;
+
                blame->commit = NULL;
-               strncpy(blame->text, line, linelen);
-               blame->text[linelen] = 0;
+               string_expand(blame->text, linelen + 1, line, opt_tab_size);
                return add_line_data(view, blame, LINE_BLAME_ID) != NULL;
        }
 }
@@ -4589,7 +4617,6 @@ status_get_diff(struct status *file, const char *buf, size_t bufsize)
 static bool
 status_run(struct view *view, const char *argv[], char status, enum line_type type)
 {
-       struct status *file = NULL;
        struct status *unmerged = NULL;
        char *buf;
        struct io io = {};
@@ -4600,6 +4627,8 @@ status_run(struct view *view, const char *argv[], char status, enum line_type ty
        add_line_data(view, NULL, type);
 
        while ((buf = io_get(&io, 0, TRUE))) {
+               struct status *file = unmerged;
+
                if (!file) {
                        file = calloc(1, sizeof(*file));
                        if (!file || !add_line_data(view, file, type))
@@ -4612,7 +4641,7 @@ status_run(struct view *view, const char *argv[], char status, enum line_type ty
                        if (status == 'A')
                                string_copy(file->old.rev, NULL_ID);
 
-               } else if (!file->status) {
+               } else if (!file->status || file == unmerged) {
                        if (!status_get_diff(file, buf, strlen(buf)))
                                goto error_out;
 
@@ -4620,21 +4649,13 @@ status_run(struct view *view, const char *argv[], char status, enum line_type ty
                        if (!buf)
                                break;
 
-                       /* Collapse all 'M'odified entries that follow a
-                        * associated 'U'nmerged entry. */
-                       if (file->status == 'U') {
-                               unmerged = file;
-
-                       } else if (unmerged) {
-                               int collapse = !strcmp(buf, unmerged->new.name);
-
+                       /* Collapse all modified entries that follow an
+                        * associated unmerged entry. */
+                       if (unmerged == file) {
+                               unmerged->status = 'U';
                                unmerged = NULL;
-                               if (collapse) {
-                                       free(file);
-                                       file = NULL;
-                                       view->lines--;
-                                       continue;
-                               }
+                       } else if (file->status == 'U') {
+                               unmerged = file;
                        }
                }
 
@@ -4780,7 +4801,7 @@ status_draw(struct view *view, struct line *line, unsigned int lineno)
 
                case LINE_STAT_NONE:
                        type = LINE_DEFAULT;
-                       text = "    (no files)";
+                       text = "  (no files)";
                        break;
 
                case LINE_STAT_HEAD:
@@ -5073,13 +5094,20 @@ status_revert(struct status *status, enum line_type type, bool has_none)
                return FALSE;
 
        } else {
+               char mode[10] = "100644";
+               const char *reset_argv[] = {
+                       "git", "update-index", "--cacheinfo", mode,
+                               status->old.rev, status->old.name, NULL
+               };
                const char *checkout_argv[] = {
                        "git", "checkout", "--", status->old.name, NULL
                };
 
                if (!prompt_yesno("Are you sure you want to overwrite any changes?"))
                        return FALSE;
-               return run_io_fg(checkout_argv, opt_cdup);
+               string_format(mode, "%o", status->old.mode);
+               return (status->status != 'U' || run_io_fg(reset_argv, opt_cdup)) &&
+                       run_io_fg(checkout_argv, opt_cdup);
        }
 }
 
@@ -5852,7 +5880,7 @@ main_read(struct view *view, char *line)
                /* FIXME: More graceful handling of titles; append "..." to
                 * shortened titles, etc. */
 
-               string_ncopy(commit->title, line, strlen(line));
+               string_expand(commit->title, sizeof(commit->title), line, 1);
                view->line[view->lines - 1].dirty = 1;
        }
 
@@ -5961,15 +5989,11 @@ static struct view_ops main_ops = {
 /*
  * Unicode / UTF-8 handling
  *
- * NOTE: Much of the following code for dealing with unicode is derived from
+ * NOTE: Much of the following code for dealing with Unicode is derived from
  * ELinks' UTF-8 code developed by Scrool <scroolik@gmail.com>. Origin file is
- * src/intl/charset.c from the utf8 branch commit elinks-0.11.0-g31f2c28.
+ * src/intl/charset.c from the UTF-8 branch commit elinks-0.11.0-g31f2c28.
  */
 
-/* I've (over)annotated a lot of code snippets because I am not entirely
- * confident that the approach taken by this small UTF-8 interface is correct.
- * --jonas */
-
 static inline int
 unicode_width(unsigned long c)
 {
@@ -6007,7 +6031,7 @@ static const unsigned char utf8_bytes[256] = {
        3,3,3,3,3,3,3,3, 3,3,3,3,3,3,3,3, 4,4,4,4,4,4,4,4, 5,5,5,5,6,6,1,1,
 };
 
-/* Decode UTF-8 multi-byte representation into a unicode character. */
+/* Decode UTF-8 multi-byte representation into a Unicode character. */
 static inline unsigned long
 utf8_to_unicode(const char *string, size_t length)
 {
@@ -6048,7 +6072,7 @@ utf8_to_unicode(const char *string, size_t length)
                unicode +=  (string[5] & 0x3f);
                break;
        default:
-               die("Invalid unicode length");
+               die("Invalid Unicode length");
        }
 
        /* Invalid characters could return the special 0xfffd value but NUL
@@ -6063,9 +6087,9 @@ utf8_to_unicode(const char *string, size_t length)
  *
  * Returns the number of bytes to output from string to satisfy max_width. */
 static size_t
-utf8_length(const char *string, int *width, size_t max_width, int *trimmed, bool reserve)
+utf8_length(const char **start, size_t skip, int *width, size_t max_width, int *trimmed, bool reserve)
 {
-       const char *start = string;
+       const char *string = *start;
        const char *end = strchr(string, '\0');
        unsigned char last_bytes = 0;
        size_t last_ucwidth = 0;
@@ -6086,11 +6110,15 @@ utf8_length(const char *string, int *width, size_t max_width, int *trimmed, bool
                 * it is a single- or double-width character. */
 
                unicode = utf8_to_unicode(string, bytes);
-               /* FIXME: Graceful handling of invalid unicode character. */
+               /* FIXME: Graceful handling of invalid Unicode character. */
                if (!unicode)
                        break;
 
                ucwidth = unicode_width(unicode);
+               if (skip > 0) {
+                       skip -= ucwidth <= skip ? ucwidth : skip;
+                       *start += bytes;
+               }
                *width  += ucwidth;
                if (*width > max_width) {
                        *trimmed = 1;
@@ -6103,11 +6131,11 @@ utf8_length(const char *string, int *width, size_t max_width, int *trimmed, bool
                }
 
                string  += bytes;
-               last_bytes = bytes;
+               last_bytes = ucwidth ? bytes : 0;
                last_ucwidth = ucwidth;
        }
 
-       return string - start;
+       return string - *start;
 }
 
 
@@ -6118,6 +6146,10 @@ utf8_length(const char *string, int *width, size_t max_width, int *trimmed, bool
 /* Whether or not the curses interface has been initialized. */
 static bool cursed = FALSE;
 
+/* Terminal hacks and workarounds. */
+static bool use_scroll_redrawwin;
+static bool use_scroll_status_wclear;
+
 /* The status window is used for polling keystrokes. */
 static WINDOW *status_win;
 
@@ -6156,6 +6188,8 @@ report(const char *msg, ...)
                va_start(args, msg);
 
                wmove(status_win, 0, 0);
+               if (view->has_scrolled && use_scroll_status_wclear)
+                       wclear(status_win);
                if (*msg) {
                        vwprintw(status_win, msg, args);
                        status_empty = FALSE;
@@ -6169,7 +6203,6 @@ report(const char *msg, ...)
        }
 
        update_view_title(view);
-       update_display_cursor(view);
 }
 
 /* Controls when nodelay should be in effect when polling user input. */
@@ -6186,6 +6219,7 @@ set_nonblocking_input(bool loading)
 static void
 init_display(void)
 {
+       const char *term;
        int x, y;
 
        /* Initialize the curses library */
@@ -6203,10 +6237,10 @@ init_display(void)
        if (!cursed)
                die("Failed to initialize curses");
 
-       nonl();         /* Tell curses not to do NL->CR/NL on output */
+       nonl();         /* Disable conversion and detect newlines from input. */
        cbreak();       /* Take input chars one at a time, no wait for \n */
        noecho();       /* Don't echo input */
-       leaveok(stdscr, TRUE);
+       leaveok(stdscr, FALSE);
 
        if (has_colors())
                init_colors();
@@ -6224,20 +6258,58 @@ init_display(void)
        if (opt_line_graphics) {
                line_graphics[LINE_GRAPHIC_VLINE] = ACS_VLINE;
        }
+
+       term = getenv("XTERM_VERSION") ? NULL : getenv("COLORTERM");
+       if (term && !strcmp(term, "gnome-terminal")) {
+               /* In the gnome-terminal-emulator, the message from
+                * scrolling up one line when impossible followed by
+                * scrolling down one line causes corruption of the
+                * status line. This is fixed by calling wclear. */
+               use_scroll_status_wclear = TRUE;
+               use_scroll_redrawwin = FALSE;
+
+       } else if (term && !strcmp(term, "xrvt-xpm")) {
+               /* No problems with full optimizations in xrvt-(unicode)
+                * and aterm. */
+               use_scroll_status_wclear = use_scroll_redrawwin = FALSE;
+
+       } else {
+               /* When scrolling in (u)xterm the last line in the
+                * scrolling direction will update slowly. */
+               use_scroll_redrawwin = TRUE;
+               use_scroll_status_wclear = FALSE;
+       }
 }
 
 static int
-get_input(bool prompting)
+get_input(int prompt_position)
 {
        struct view *view;
-       int i, key;
+       int i, key, cursor_y, cursor_x;
 
-       if (prompting)
+       if (prompt_position)
                input_mode = TRUE;
 
-       while (true) {
-               foreach_view (view, i)
+       while (TRUE) {
+               foreach_view (view, i) {
                        update_view(view);
+                       if (view_is_displayed(view) && view->has_scrolled &&
+                           use_scroll_redrawwin)
+                               redrawwin(view->win);
+                       view->has_scrolled = FALSE;
+               }
+
+               /* Update the cursor position. */
+               if (prompt_position) {
+                       getbegyx(status_win, cursor_y, cursor_x);
+                       cursor_x = prompt_position;
+               } else {
+                       view = display[current_view];
+                       getbegyx(view->win, cursor_y, cursor_x);
+                       cursor_x = view->width - 1;
+                       cursor_y += view->lineno - view->offset;
+               }
+               setsyx(cursor_y, cursor_x);
 
                /* Refresh, accept single keystroke of input */
                doupdate();
@@ -6252,8 +6324,6 @@ get_input(bool prompting)
 
                        getmaxyx(stdscr, height, width);
 
-                       /* Resize the status view first so the cursor is
-                        * placed properly. */
                        wresize(status_win, 1, width);
                        mvwin(status_win, height - 1, 0);
                        wnoutrefresh(status_win);
@@ -6282,7 +6352,7 @@ prompt_input(const char *prompt, input_handler handler, void *data)
                mvwprintw(status_win, 0, 0, "%s%.*s", prompt, pos, buf);
                wclrtoeol(status_win);
 
-               key = get_input(TRUE);
+               key = get_input(pos + 1);
                switch (key) {
                case KEY_RETURN:
                case KEY_ENTER:
@@ -6362,17 +6432,6 @@ read_prompt(const char *prompt)
  * Repository properties
  */
 
-static int
-git_properties(const char **argv, const char *separators,
-              int (*read_property)(char *, size_t, char *, size_t))
-{
-       struct io io = {};
-
-       if (init_io_rd(&io, argv, NULL, FORMAT_NONE))
-               return read_properties(&io, separators, read_property);
-       return ERR;
-}
-
 static struct ref *refs = NULL;
 static size_t refs_alloc = 0;
 static size_t refs_size = 0;
@@ -6496,7 +6555,7 @@ read_ref(char *id, size_t idlen, char *name, size_t namelen)
        }
 
        if (check_replace && !strcmp(name, refs[refs_size - 1].name)) {
-               /* it's an annotated tag, replace the previous sha1 with the
+               /* 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 before the commit id
                 * it points to. */
@@ -6547,7 +6606,23 @@ load_refs(void)
        while (id_refs_size > 0)
                free(id_refs[--id_refs_size]);
 
-       return git_properties(ls_remote_argv, "\t", read_ref);
+       return run_io_load(ls_remote_argv, "\t", read_ref);
+}
+
+static void
+set_repo_config_option(char *name, char *value, int (*cmd)(int, const char **))
+{
+       const char *argv[SIZEOF_ARG] = { name, "=" };
+       int argc = 1 + (cmd == option_set_command);
+       int error = ERR;
+
+       if (!argv_from_string(argv, &argc, value))
+               config_msg = "Too many option arguments";
+       else
+               error = cmd(argc, argv);
+
+       if (error == ERR)
+               warn("Option 'tig.%s': %s", name, config_msg);
 }
 
 static int
@@ -6559,6 +6634,15 @@ read_repo_config_option(char *name, size_t namelen, char *value, size_t valuelen
        if (!strcmp(name, "core.editor"))
                string_ncopy(opt_editor, value, valuelen);
 
+       if (!prefixcmp(name, "tig.color."))
+               set_repo_config_option(name + 10, value, option_color_command);
+
+       else if (!prefixcmp(name, "tig.bind."))
+               set_repo_config_option(name + 9, value, option_bind_command);
+
+       else if (!prefixcmp(name, "tig."))
+               set_repo_config_option(name + 4, value, option_set_command);
+
        /* branch.<head>.remote */
        if (*opt_head &&
            !strncmp(name, "branch.", 7) &&
@@ -6589,7 +6673,7 @@ load_git_config(void)
 {
        const char *config_list_argv[] = { "git", GIT_CONFIG, "--list", NULL };
 
-       return git_properties(config_list_argv, "=", read_repo_config_option);
+       return run_io_load(config_list_argv, "=", read_repo_config_option);
 }
 
 static int
@@ -6636,45 +6720,7 @@ load_repo_info(void)
                }
        }
 
-       return git_properties(rev_parse_argv, "=", read_repo_info);
-}
-
-static int
-read_properties(struct io *io, const char *separators,
-               int (*read_property)(char *, size_t, char *, size_t))
-{
-       char *name;
-       int state = OK;
-
-       if (!start_io(io))
-               return ERR;
-
-       while (state == OK && (name = io_get(io, '\n', TRUE))) {
-               char *value;
-               size_t namelen;
-               size_t valuelen;
-
-               name = chomp_string(name);
-               namelen = strcspn(name, separators);
-
-               if (name[namelen]) {
-                       name[namelen] = 0;
-                       value = chomp_string(name + namelen + 1);
-                       valuelen = strlen(value);
-
-               } else {
-                       value = "";
-                       valuelen = 0;
-               }
-
-               state = read_property(name, namelen, value, valuelen);
-       }
-
-       if (state != ERR && io_error(io))
-               state = ERR;
-       done_io(io);
-
-       return state;
+       return run_io_load(rev_parse_argv, "=", read_repo_info);
 }
 
 
@@ -6682,6 +6728,19 @@ read_properties(struct io *io, const char *separators,
  * Main
  */
 
+static const char usage[] =
+"tig " TIG_VERSION " (" __DATE__ ")\n"
+"\n"
+"Usage: tig        [options] [revs] [--] [paths]\n"
+"   or: tig show   [options] [revs] [--] [paths]\n"
+"   or: tig blame  [rev] path\n"
+"   or: tig status\n"
+"   or: tig <      [git command output]\n"
+"\n"
+"Options:\n"
+"  -v, --version   Show version and exit\n"
+"  -h, --help      Show help message and exit";
+
 static void __NORETURN
 quit(int sig)
 {
@@ -6719,12 +6778,90 @@ warn(const char *msg, ...)
        va_end(args);
 }
 
+static enum request
+parse_options(int argc, const char *argv[])
+{
+       enum request request = REQ_VIEW_MAIN;
+       const char *subcommand;
+       bool seen_dashdash = FALSE;
+       /* XXX: This is vulnerable to the user overriding options
+        * required for the main view parser. */
+       const char *custom_argv[SIZEOF_ARG] = {
+               "git", "log", "--no-color", "--pretty=raw", "--parents",
+                       "--topo-order", NULL
+       };
+       int i, j = 6;
+
+       if (!isatty(STDIN_FILENO)) {
+               io_open(&VIEW(REQ_VIEW_PAGER)->io, "");
+               return REQ_VIEW_PAGER;
+       }
+
+       if (argc <= 1)
+               return REQ_NONE;
+
+       subcommand = argv[1];
+       if (!strcmp(subcommand, "status")) {
+               if (argc > 2)
+                       warn("ignoring arguments after `%s'", subcommand);
+               return REQ_VIEW_STATUS;
+
+       } else if (!strcmp(subcommand, "blame")) {
+               if (argc <= 2 || argc > 4)
+                       die("invalid number of options to blame\n\n%s", usage);
+
+               i = 2;
+               if (argc == 4) {
+                       string_ncopy(opt_ref, argv[i], strlen(argv[i]));
+                       i++;
+               }
+
+               string_ncopy(opt_file, argv[i], strlen(argv[i]));
+               return REQ_VIEW_BLAME;
+
+       } else if (!strcmp(subcommand, "show")) {
+               request = REQ_VIEW_DIFF;
+
+       } else {
+               subcommand = NULL;
+       }
+
+       if (subcommand) {
+               custom_argv[1] = subcommand;
+               j = 2;
+       }
+
+       for (i = 1 + !!subcommand; i < argc; i++) {
+               const char *opt = argv[i];
+
+               if (seen_dashdash || !strcmp(opt, "--")) {
+                       seen_dashdash = TRUE;
+
+               } else if (!strcmp(opt, "-v") || !strcmp(opt, "--version")) {
+                       printf("tig version %s\n", TIG_VERSION);
+                       quit(0);
+
+               } else if (!strcmp(opt, "-h") || !strcmp(opt, "--help")) {
+                       printf("%s\n", usage);
+                       quit(0);
+               }
+
+               custom_argv[j++] = opt;
+               if (j >= ARRAY_SIZE(custom_argv))
+                       die("command too long");
+       }
+
+       if (!prepare_update(VIEW(request), custom_argv, NULL, FORMAT_NONE))                                                                        
+               die("Failed to format arguments"); 
+
+       return request;
+}
+
 int
 main(int argc, const char *argv[])
 {
-       const char **run_argv = NULL;
+       enum request request = parse_options(argc, argv);
        struct view *view;
-       enum request request;
        size_t i;
 
        signal(SIGINT, quit);
@@ -6744,10 +6881,6 @@ main(int argc, const char *argv[])
        if (load_git_config() == ERR)
                die("Failed to load repo config.");
 
-       request = parse_options(argc, argv, &run_argv);
-       if (request == REQ_NONE)
-               return 0;
-
        /* Require a git repository unless when running in pager mode. */
        if (!opt_git_dir[0] && request != REQ_VIEW_PAGER)
                die("Not a git repository");
@@ -6769,17 +6902,12 @@ main(int argc, const char *argv[])
 
        init_display();
 
-       if (request == REQ_VIEW_PAGER || run_argv) {
-               if (request == REQ_VIEW_PAGER)
-                       io_open(&VIEW(request)->io, "");
-               else if (!prepare_update(VIEW(request), run_argv, NULL, FORMAT_NONE))
-                       die("Failed to format arguments");
+       if (request != REQ_NONE)
                open_view(NULL, request, OPEN_PREPARED);
-               request = REQ_NONE;
-       }
+       request = request == REQ_NONE ? REQ_VIEW_MAIN : REQ_NONE;
 
        while (view_driver(display[current_view], request)) {
-               int key = get_input(FALSE);
+               int key = get_input(0);
 
                view = display[current_view];
                request = get_keybinding(view->keymap, key);
@@ -6821,6 +6949,10 @@ main(int argc, const char *argv[])
 
                        if (search)
                                string_ncopy(opt_search, search, strlen(search));
+                       else if (*opt_search)
+                               request = request == REQ_SEARCH ?
+                                       REQ_FIND_NEXT :
+                                       REQ_FIND_PREV;
                        else
                                request = REQ_NONE;
                        break;