Code

Refactor the int_map interface into new enum_map interface
[tig.git] / tig.c
diff --git a/tig.c b/tig.c
index bd9a7a4bf0852cd23413adf32c5eb3282beaadde..c5e3380bbd5a267f38639a02647b66e76f14d29e 100644 (file)
--- a/tig.c
+++ b/tig.c
@@ -121,7 +121,7 @@ static size_t utf8_length(const char **string, size_t col, int *width, size_t ma
 #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
@@ -148,29 +148,6 @@ enum format_flags {
 
 static bool format_argv(const char *dst[], const char *src[], enum format_flags flags);
 
-struct int_map {
-       const char *name;
-       int namelen;
-       int value;
-};
-
-static int
-set_from_int_map(struct int_map *map, size_t map_size,
-                int *value, const char *name, int namelen)
-{
-
-       int i;
-
-       for (i = 0; i < map_size; i++)
-               if (namelen == map[i].namelen &&
-                   !strncasecmp(name, map[i].name, namelen)) {
-                       *value = map[i].value;
-                       return OK;
-               }
-
-       return ERR;
-}
-
 enum input_status {
        INPUT_OK,
        INPUT_SKIP,
@@ -305,6 +282,33 @@ string_enum_compare(const char *str1, const char *str2, int len)
        return 0;
 }
 
+struct enum_map {
+       const char *name;
+       int namelen;
+       int value;
+};
+
+#define ENUM_MAP(name, value) { name, STRING_SIZE(name), value }
+
+static bool
+map_enum_do(struct enum_map *map, size_t map_size, int *value, const char *name)
+{
+       size_t namelen = strlen(name);
+       int i;
+
+       for (i = 0; i < map_size; i++)
+               if (namelen == map[i].namelen &&
+                   !string_enum_compare(name, map[i].name, namelen)) {
+                       *value = map[i].value;
+                       return TRUE;
+               }
+
+       return FALSE;
+}
+
+#define map_enum(attr, map, name) \
+       map_enum_do(map, ARRAY_SIZE(map), attr, name)
+
 #define prefixcmp(str1, str2) \
        strncmp(str1, str2, STRING_SIZE(str2))
 
@@ -927,6 +931,7 @@ 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), \
@@ -938,8 +943,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), \
@@ -1139,14 +1143,13 @@ enum keymap {
 #undef KEYMAP_
 };
 
-static struct int_map keymap_table[] = {
-#define KEYMAP_(name) { #name, STRING_SIZE(#name), KEYMAP_##name }
+static struct enum_map keymap_table[] = {
+#define KEYMAP_(name) ENUM_MAP(#name, KEYMAP_##name)
        KEYMAP_INFO
 #undef KEYMAP_
 };
 
-#define set_keymap(map, name) \
-       set_from_int_map(keymap_table, ARRAY_SIZE(keymap_table), map, name, strlen(name))
+#define set_keymap(map, name) map_enum(map, keymap_table, name)
 
 struct keybinding_table {
        struct keybinding *data;
@@ -1357,8 +1360,8 @@ add_builtin_run_requests(void)
  * User config file handling.
  */
 
-static struct int_map color_map[] = {
-#define COLOR_MAP(name) { #name, STRING_SIZE(#name), COLOR_##name }
+static struct enum_map color_map[] = {
+#define COLOR_MAP(name) ENUM_MAP(#name, COLOR_##name)
        COLOR_MAP(DEFAULT),
        COLOR_MAP(BLACK),
        COLOR_MAP(BLUE),
@@ -1370,11 +1373,8 @@ static struct int_map color_map[] = {
        COLOR_MAP(YELLOW),
 };
 
-#define set_color(color, name) \
-       set_from_int_map(color_map, ARRAY_SIZE(color_map), color, name, strlen(name))
-
-static struct int_map attr_map[] = {
-#define ATTR_MAP(name) { #name, STRING_SIZE(#name), A_##name }
+static struct enum_map attr_map[] = {
+#define ATTR_MAP(name) ENUM_MAP(#name, A_##name)
        ATTR_MAP(NORMAL),
        ATTR_MAP(BLINK),
        ATTR_MAP(BOLD),
@@ -1384,14 +1384,14 @@ static struct int_map attr_map[] = {
        ATTR_MAP(UNDERLINE),
 };
 
-#define set_attribute(attr, name) \
-       set_from_int_map(attr_map, ARRAY_SIZE(attr_map), attr, name, strlen(name))
+#define set_color(color, name)         map_enum(color, color_map, name)
+#define set_attribute(attr, name)      map_enum(attr, attr_map, name)
 
 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[])
 {
@@ -1419,13 +1419,13 @@ option_color_command(int argc, const char *argv[])
                }
        }
 
-       if (set_color(&info->fg, argv[1]) == ERR ||
-           set_color(&info->bg, argv[2]) == ERR) {
+       if (!set_color(&info->fg, argv[1]) ||
+           !set_color(&info->bg, argv[2])) {
                config_msg = "Unknown color";
                return ERR;
        }
 
-       if (argc == 4 && set_attribute(&info->attr, argv[3]) == ERR) {
+       if (argc == 4 && !set_attribute(&info->attr, argv[3])) {
                config_msg = "Unknown attribute";
                return ERR;
        }
@@ -1433,18 +1433,40 @@ option_color_command(int argc, const char *argv[])
        return OK;
 }
 
-static bool parse_bool(const char *s)
+static int parse_bool(bool *opt, const char *arg)
 {
-       return (!strcmp(s, "1") || !strcmp(s, "true") ||
-               !strcmp(s, "yes")) ? TRUE : FALSE;
+       *opt = (!strcmp(arg, "1") || !strcmp(arg, "true") || !strcmp(arg, "yes"))
+               ? TRUE : FALSE;
+       return OK;
+}
+
+static int
+parse_int(int *opt, const char *arg, int min, int max)
+{
+       int value = atoi(arg);
+
+       if (min <= value && value <= max)
+               *opt = value;
+       return OK;
 }
 
 static int
-parse_int(const char *s, int default_value, int min, int max)
+parse_string(char *opt, const char *arg, size_t optsize)
 {
-       int value = atoi(s);
+       int arglen = strlen(arg);
 
-       return (value < min || value > max) ? default_value : value;
+       switch (arg[0]) {
+       case '\"':
+       case '\'':
+               if (arglen == 1 || arg[arglen - 1] != arg[0]) {
+                       config_msg = "Unmatched quotation";
+                       return ERR;
+               }
+               arg += 1; arglen -= 2;
+       default:
+               string_ncopy_do(opt, optsize, arg, strlen(arg));
+               return OK;
+       }
 }
 
 /* Wants: name = value */
@@ -1461,68 +1483,35 @@ option_set_command(int argc, const char *argv[])
                return ERR;
        }
 
-       if (!strcmp(argv[0], "show-author")) {
-               opt_author = parse_bool(argv[2]);
-               return OK;
-       }
+       if (!strcmp(argv[0], "show-author"))
+               return parse_bool(&opt_author, argv[2]);
 
-       if (!strcmp(argv[0], "show-date")) {
-               opt_date = parse_bool(argv[2]);
-               return OK;
-       }
+       if (!strcmp(argv[0], "show-date"))
+               return parse_bool(&opt_date, argv[2]);
 
-       if (!strcmp(argv[0], "show-rev-graph")) {
-               opt_rev_graph = parse_bool(argv[2]);
-               return OK;
-       }
+       if (!strcmp(argv[0], "show-rev-graph"))
+               return parse_bool(&opt_rev_graph, argv[2]);
 
-       if (!strcmp(argv[0], "show-refs")) {
-               opt_show_refs = parse_bool(argv[2]);
-               return OK;
-       }
+       if (!strcmp(argv[0], "show-refs"))
+               return parse_bool(&opt_show_refs, argv[2]);
 
-       if (!strcmp(argv[0], "show-line-numbers")) {
-               opt_line_number = parse_bool(argv[2]);
-               return OK;
-       }
+       if (!strcmp(argv[0], "show-line-numbers"))
+               return parse_bool(&opt_line_number, argv[2]);
 
-       if (!strcmp(argv[0], "line-graphics")) {
-               opt_line_graphics = parse_bool(argv[2]);
-               return OK;
-       }
+       if (!strcmp(argv[0], "line-graphics"))
+               return parse_bool(&opt_line_graphics, argv[2]);
 
-       if (!strcmp(argv[0], "line-number-interval")) {
-               opt_num_interval = parse_int(argv[2], opt_num_interval, 1, 1024);
-               return OK;
-       }
+       if (!strcmp(argv[0], "line-number-interval"))
+               return parse_int(&opt_num_interval, argv[2], 1, 1024);
 
-       if (!strcmp(argv[0], "author-width")) {
-               opt_author_cols = parse_int(argv[2], opt_author_cols, 0, 1024);
-               return OK;
-       }
+       if (!strcmp(argv[0], "author-width"))
+               return parse_int(&opt_author_cols, argv[2], 0, 1024);
 
-       if (!strcmp(argv[0], "tab-size")) {
-               opt_tab_size = parse_int(argv[2], opt_tab_size, 1, 1024);
-               return OK;
-       }
+       if (!strcmp(argv[0], "tab-size"))
+               return parse_int(&opt_tab_size, argv[2], 1, 1024);
 
-       if (!strcmp(argv[0], "commit-encoding")) {
-               const char *arg = argv[2];
-               int arglen = strlen(arg);
-
-               switch (arg[0]) {
-               case '"':
-               case '\'':
-                       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;
-               }
-       }
+       if (!strcmp(argv[0], "commit-encoding"))
+               return parse_string(opt_encoding, argv[2], sizeof(opt_encoding));
 
        config_msg = "Unknown variable name";
        return ERR;
@@ -1656,7 +1645,7 @@ 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;
 
@@ -1745,7 +1734,7 @@ struct view {
 
        /* 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. */
@@ -1784,7 +1773,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);
@@ -2035,6 +2024,27 @@ draw_author(struct view *view, const char *author)
        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
 draw_view_line(struct view *view, unsigned int lineno)
 {
@@ -3760,14 +3770,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 ")
@@ -3839,7 +3846,7 @@ tree_read_date(struct view *view, char *text, bool *read_date)
                struct io io = {};
 
                if (!view->lines) {
-                       tree_entry(view, LINE_TREE_PARENT, opt_path, NULL, NULL);
+                       tree_entry(view, LINE_TREE_HEAD, opt_path, NULL, NULL);
                        report("Tree is empty");
                        return TRUE;
                }
@@ -3911,7 +3918,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 ... */
@@ -3962,27 +3969,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))
@@ -4059,7 +4050,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();
@@ -4647,8 +4638,8 @@ 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. */
+                       /* Collapse all modified entries that follow an
+                        * associated unmerged entry. */
                        if (unmerged == file) {
                                unmerged->status = 'U';
                                unmerged = NULL;
@@ -5987,15 +5978,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)
 {
@@ -6033,7 +6020,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)
 {
@@ -6074,7 +6061,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
@@ -6112,7 +6099,7 @@ utf8_length(const char **start, size_t skip, int *width, size_t max_width, int *
                 * 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;
 
@@ -6239,7 +6226,7 @@ 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, FALSE);
@@ -6557,7 +6544,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. */
@@ -6611,6 +6598,22 @@ load_refs(void)
        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
 read_repo_config_option(char *name, size_t namelen, char *value, size_t valuelen)
 {
@@ -6620,6 +6623,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) &&