X-Git-Url: https://git.distorted.org.uk/~mdw/tig/blobdiff_plain/70ea81756390302e6dd816b3c2de87a614879d0f..f4de14c5762fe2beef8633a254c470a37376b4c2:/tig.c diff --git a/tig.c b/tig.c index a1a6a80..ee9d706 100644 --- a/tig.c +++ b/tig.c @@ -103,6 +103,8 @@ static size_t utf8_length(const char *string, size_t max_width, int *trimmed, bo #define SCALE_SPLIT_VIEW(height) ((height) * 2 / 3) +#define NULL_ID "0000000000000000000000000000000000000000" + #ifndef GIT_CONFIG #define GIT_CONFIG "git config" #endif @@ -329,6 +331,7 @@ sq_quote(char buf[SIZEOF_STR], size_t bufsize, const char *src) REQ_(PREVIOUS, "Move to previous"), \ REQ_(VIEW_NEXT, "Move focus to next view"), \ REQ_(REFRESH, "Reload and refresh"), \ + REQ_(MAXIMIZE, "Maximize the current view"), \ REQ_(VIEW_CLOSE, "Close the current view"), \ REQ_(QUIT, "Close all views and quit"), \ \ @@ -444,6 +447,7 @@ 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 bool opt_no_head = TRUE; static FILE *opt_pipe = NULL; static char opt_encoding[20] = "UTF-8"; static bool opt_utf8 = TRUE; @@ -463,6 +467,12 @@ parse_options(int argc, char *argv[]) bool seen_dashdash = FALSE; int i; + if (!isatty(STDIN_FILENO)) { + opt_request = REQ_VIEW_PAGER; + opt_pipe = stdin; + return TRUE; + } + if (argc <= 1) return TRUE; @@ -521,7 +531,7 @@ parse_options(int argc, char *argv[]) return FALSE; } else if (!strcmp(opt, "-h") || !strcmp(opt, "--help")) { - printf(usage); + printf("%s\n", usage); return FALSE; } @@ -531,12 +541,6 @@ parse_options(int argc, char *argv[]) die("command too long"); } - if (!isatty(STDIN_FILENO)) { - opt_request = REQ_VIEW_PAGER; - opt_pipe = stdin; - buf_size = 0; - } - opt_cmd[buf_size] = 0; return TRUE; @@ -579,12 +583,12 @@ LINE(ACKED, " Acked-by", COLOR_YELLOW, COLOR_DEFAULT, 0), \ LINE(DEFAULT, "", COLOR_DEFAULT, COLOR_DEFAULT, A_NORMAL), \ LINE(CURSOR, "", COLOR_WHITE, COLOR_GREEN, A_BOLD), \ LINE(STATUS, "", COLOR_GREEN, COLOR_DEFAULT, 0), \ +LINE(DELIMITER, "", COLOR_MAGENTA, COLOR_DEFAULT, 0), \ LINE(TITLE_BLUR, "", COLOR_WHITE, COLOR_BLUE, 0), \ LINE(TITLE_FOCUS, "", COLOR_WHITE, COLOR_BLUE, A_BOLD), \ LINE(MAIN_DATE, "", COLOR_BLUE, COLOR_DEFAULT, 0), \ LINE(MAIN_AUTHOR, "", COLOR_GREEN, COLOR_DEFAULT, 0), \ LINE(MAIN_COMMIT, "", COLOR_DEFAULT, COLOR_DEFAULT, 0), \ -LINE(MAIN_DELIM, "", COLOR_MAGENTA, COLOR_DEFAULT, 0), \ LINE(MAIN_TAG, "", COLOR_MAGENTA, COLOR_DEFAULT, A_BOLD), \ LINE(MAIN_LOCAL_TAG,"", COLOR_MAGENTA, COLOR_DEFAULT, A_BOLD), \ LINE(MAIN_REMOTE, "", COLOR_YELLOW, COLOR_DEFAULT, A_BOLD), \ @@ -593,6 +597,7 @@ LINE(MAIN_HEAD, "", COLOR_RED, COLOR_DEFAULT, A_BOLD), \ LINE(MAIN_REVGRAPH,"", COLOR_MAGENTA, COLOR_DEFAULT, 0), \ LINE(TREE_DIR, "", COLOR_DEFAULT, COLOR_DEFAULT, A_NORMAL), \ LINE(TREE_FILE, "", COLOR_DEFAULT, COLOR_DEFAULT, A_NORMAL), \ +LINE(STAT_HEAD, "", COLOR_YELLOW, COLOR_DEFAULT, 0), \ LINE(STAT_SECTION, "", COLOR_CYAN, COLOR_DEFAULT, 0), \ LINE(STAT_NONE, "", COLOR_DEFAULT, COLOR_DEFAULT, 0), \ LINE(STAT_STAGED, "", COLOR_MAGENTA, COLOR_DEFAULT, 0), \ @@ -649,8 +654,9 @@ get_line_attr(enum line_type type) } static struct line_info * -get_line_info(char *name, int namelen) +get_line_info(char *name) { + size_t namelen = strlen(name); enum line_type type; for (type = 0; type < ARRAY_SIZE(line_info); type++) @@ -725,6 +731,7 @@ static struct keybinding default_keybindings[] = { { KEY_UP, REQ_PREVIOUS }, { KEY_DOWN, REQ_NEXT }, { 'R', REQ_REFRESH }, + { 'M', REQ_MAXIMIZE }, /* Cursor navigation */ { 'k', REQ_MOVE_UP }, @@ -1046,10 +1053,15 @@ option_color_command(int argc, char *argv[]) return ERR; } - info = get_line_info(argv[0], strlen(argv[0])); + info = get_line_info(argv[0]); if (!info) { - config_msg = "Unknown color name"; - return ERR; + if (!string_enum_compare(argv[0], "main-delim", strlen("main-delim"))) { + info = get_line_info("delimiter"); + + } else { + config_msg = "Unknown color name"; + return ERR; + } } if (set_color(&info->fg, argv[1]) == ERR || @@ -1347,6 +1359,7 @@ struct view { struct view_ops *ops; /* View operations */ 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 */ @@ -1406,27 +1419,28 @@ static struct view_ops help_ops; static struct view_ops status_ops; static struct view_ops stage_ops; -#define VIEW_STR(name, cmd, env, ref, ops, map) \ - { name, cmd, #env, ref, ops, map} +#define VIEW_STR(name, cmd, env, ref, ops, map, git) \ + { name, cmd, #env, ref, ops, map, git } -#define VIEW_(id, name, ops, ref) \ - VIEW_STR(name, TIG_##id##_CMD, TIG_##id##_CMD, ref, ops, KEYMAP_##id) +#define VIEW_(id, name, ops, git, ref) \ + VIEW_STR(name, TIG_##id##_CMD, TIG_##id##_CMD, ref, ops, KEYMAP_##id, git) static struct view views[] = { - VIEW_(MAIN, "main", &main_ops, ref_head), - VIEW_(DIFF, "diff", &pager_ops, ref_commit), - VIEW_(LOG, "log", &pager_ops, ref_head), - VIEW_(TREE, "tree", &tree_ops, ref_commit), - VIEW_(BLOB, "blob", &blob_ops, ref_blob), - VIEW_(BLAME, "blame", &blame_ops, ref_commit), - VIEW_(HELP, "help", &help_ops, ""), - VIEW_(PAGER, "pager", &pager_ops, "stdin"), - VIEW_(STATUS, "status", &status_ops, ""), - VIEW_(STAGE, "stage", &stage_ops, ""), + 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_(TREE, "tree", &tree_ops, TRUE, ref_commit), + VIEW_(BLOB, "blob", &blob_ops, TRUE, ref_blob), + VIEW_(BLAME, "blame", &blame_ops, TRUE, ref_commit), + VIEW_(HELP, "help", &help_ops, FALSE, ""), + VIEW_(PAGER, "pager", &pager_ops, FALSE, "stdin"), + VIEW_(STATUS, "status", &status_ops, TRUE, ""), + VIEW_(STAGE, "stage", &stage_ops, TRUE, ""), }; -#define VIEW(req) (&views[(req) - REQ_OFFSET - 1]) +#define VIEW(req) (&views[(req) - REQ_OFFSET - 1]) +#define VIEW_REQ(view) ((view) - views + REQ_OFFSET + 1) #define foreach_view(view, i) \ for (i = 0; i < ARRAY_SIZE(views) && (view = &views[i]); i++) @@ -1436,7 +1450,7 @@ static struct view views[] = { static int draw_text(struct view *view, const char *string, int max_len, - bool use_tilde, int tilde_attr) + bool use_tilde, bool selected) { int len = 0; int trimmed = FALSE; @@ -1459,8 +1473,8 @@ draw_text(struct view *view, const char *string, int max_len, waddnstr(view->win, string, len); if (trimmed && use_tilde) { - if (tilde_attr != -1) - wattrset(view->win, tilde_attr); + if (!selected) + wattrset(view->win, get_line_attr(LINE_DELIMITER)); waddch(view->win, '~'); len++; } @@ -1468,6 +1482,43 @@ draw_text(struct view *view, const char *string, int max_len, return len; } +static int +draw_lineno(struct view *view, unsigned int lineno, int max, bool selected) +{ + static char fmt[] = "%1ld"; + char number[10] = " "; + int max_number = MIN(view->digits, STRING_SIZE(number)); + bool showtrimmed = FALSE; + int col; + + lineno += view->offset + 1; + if (lineno == 1 || (lineno % opt_num_interval) == 0) { + if (view->digits <= 9) + fmt[1] = '0' + view->digits; + + if (!string_format(number, fmt, lineno)) + number[0] = 0; + showtrimmed = TRUE; + } + + if (max < max_number) + max_number = max; + + col = draw_text(view, number, max_number, showtrimmed, selected); + if (col < max) { + if (!selected) + wattrset(view->win, A_NORMAL); + waddch(view->win, ACS_VLINE); + col++; + } + if (col < max) { + waddch(view->win, ' '); + col++; + } + + return col; +} + static bool draw_view_line(struct view *view, unsigned int lineno) { @@ -2175,7 +2226,6 @@ update_view(struct view *view) * might have rearranged things. */ redraw_view(view); - } else if (redraw_from >= 0) { /* If this is an incremental update, redraw the previous line * since for commits some members could have changed when @@ -2269,15 +2319,8 @@ open_view(struct view *prev, enum request request, enum open_flags flags) return; } - if (view->ops->open) { - if (!view->ops->open(view)) { - report("Failed to load %s view", view->name); - return; - } - - } else if ((reload || strcmp(view->vid, view->id)) && - !begin_update(view)) { - report("Failed to load %s view", view->name); + if (view->git_dir && !opt_git_dir[0]) { + report("The %s view is disabled in pager view", view->name); return; } @@ -2298,6 +2341,18 @@ open_view(struct view *prev, enum request request, enum open_flags flags) (nviews == 1 && base_view != display[0])) resize_display(); + if (view->ops->open) { + if (!view->ops->open(view)) { + report("Failed to load %s view", view->name); + return; + } + + } else if ((reload || strcmp(view->vid, view->id)) && + !begin_update(view)) { + report("Failed to load %s view", view->name); + return; + } + if (split && prev->lineno - prev->offset >= prev->height) { /* Take the title line into account. */ int lines = prev->lineno - prev->offset - prev->height + 1; @@ -2572,6 +2627,11 @@ view_driver(struct view *view, enum request request) report("Refreshing is not yet supported for the %s view", view->name); break; + case REQ_MAXIMIZE: + if (displayed_views() == 2) + open_view(view, VIEW_REQ(view), OPEN_DEFAULT); + break; + case REQ_TOGGLE_LINENO: opt_line_number = !opt_line_number; redraw_display(); @@ -2677,39 +2737,35 @@ view_driver(struct view *view, enum request request) static bool pager_draw(struct view *view, struct line *line, unsigned int lineno, bool selected) { + static char spaces[] = " "; char *text = line->data; enum line_type type = line->type; - int attr; + int attr = A_NORMAL; + int col = 0; wmove(view->win, lineno, 0); if (selected) { type = LINE_CURSOR; wchgat(view->win, -1, 0, type, NULL); + attr = get_line_attr(type); } - - attr = get_line_attr(type); wattrset(view->win, attr); - if (opt_line_number || opt_tab_size < TABSIZE) { - static char spaces[] = " "; - int col_offset = 0, col = 0; - - if (opt_line_number) { - unsigned long real_lineno = view->offset + lineno + 1; - - if (real_lineno == 1 || - (real_lineno % opt_num_interval) == 0) { - wprintw(view->win, "%.*d", view->digits, real_lineno); + if (opt_line_number) { + col += draw_lineno(view, lineno, view->width, selected); + if (col >= view->width) + return TRUE; + } - } else { - waddnstr(view->win, spaces, - MIN(view->digits, STRING_SIZE(spaces))); - } - waddstr(view->win, ": "); - col_offset = view->digits + 2; - } + if (!selected) { + attr = get_line_attr(type); + wattrset(view->win, attr); + } + if (opt_tab_size < TABSIZE) { + int col_offset = col; + col = 0; while (text && col_offset + col < view->width) { int cols_max = view->width - col_offset - col; char *pos = text; @@ -2731,9 +2787,7 @@ pager_draw(struct view *view, struct line *line, unsigned int lineno, bool selec } } else { - int tilde_attr = get_line_attr(LINE_MAIN_DELIM); - - draw_text(view, text, view->width, TRUE, tilde_attr); + draw_text(view, text, view->width - col, TRUE, selected); } return TRUE; @@ -3407,9 +3461,9 @@ parse_blame_commit(struct view *view, char *text, int *blamed) blame = line->data; blame->commit = commit; + blame->header = !group; line->dirty = 1; } - blame->header = 1; return commit; } @@ -3519,7 +3573,6 @@ blame_read(struct view *view, char *line) static bool blame_draw(struct view *view, struct line *line, unsigned int lineno, bool selected) { - int tilde_attr = -1; struct blame *blame = line->data; int col = 0; @@ -3530,7 +3583,6 @@ blame_draw(struct view *view, struct line *line, unsigned int lineno, bool selec wchgat(view->win, -1, 0, LINE_CURSOR, NULL); } else { wattrset(view->win, A_NORMAL); - tilde_attr = get_line_attr(LINE_MAIN_DELIM); } if (opt_date) { @@ -3543,8 +3595,8 @@ blame_draw(struct view *view, struct line *line, unsigned int lineno, bool selec int timelen; timelen = strftime(buf, sizeof(buf), DATE_FORMAT, &blame->commit->time); - n = draw_text(view, buf, view->width - col, FALSE, tilde_attr); - draw_text(view, " ", view->width - col - n, FALSE, tilde_attr); + n = draw_text(view, buf, view->width - col, FALSE, selected); + draw_text(view, " ", view->width - col - n, FALSE, selected); } col += DATE_COLS; @@ -3559,7 +3611,7 @@ blame_draw(struct view *view, struct line *line, unsigned int lineno, bool selec if (!selected) wattrset(view->win, get_line_attr(LINE_MAIN_AUTHOR)); if (blame->commit) - draw_text(view, blame->commit->author, max, TRUE, tilde_attr); + draw_text(view, blame->commit->author, max, TRUE, selected); col += AUTHOR_COLS; if (col >= view->width) return TRUE; @@ -3580,44 +3632,14 @@ blame_draw(struct view *view, struct line *line, unsigned int lineno, bool selec } { - unsigned long real_lineno = view->offset + lineno + 1; - char number[10] = " "; - int max = MIN(view->digits, STRING_SIZE(number)); - bool showtrimmed = FALSE; - - if (real_lineno == 1 || - (real_lineno % opt_num_interval) == 0) { - char fmt[] = "%1ld"; - - if (view->digits <= 9) - fmt[1] = '0' + view->digits; - - if (!string_format(number, fmt, real_lineno)) - number[0] = 0; - showtrimmed = TRUE; - } - - if (max > view->width - col) - max = view->width - col; if (!selected) wattrset(view->win, get_line_attr(LINE_BLAME_LINENO)); - col += draw_text(view, number, max, showtrimmed, tilde_attr); + col += draw_lineno(view, lineno, view->width - col, selected); if (col >= view->width) return TRUE; } - if (!selected) - wattrset(view->win, A_NORMAL); - - if (col >= view->width) - return TRUE; - waddch(view->win, ACS_VLINE); - col++; - if (col >= view->width) - return TRUE; - waddch(view->win, ' '); - col++; - col += draw_text(view, blame->text, view->width - col, TRUE, tilde_attr); + col += draw_text(view, blame->text, view->width - col, TRUE, selected); return TRUE; } @@ -3635,7 +3657,7 @@ blame_request(struct view *view, enum request request, struct line *line) break; } - if (!strcmp(blame->commit->id, "0000000000000000000000000000000000000000")) { + if (!strcmp(blame->commit->id, NULL_ID)) { char path[SIZEOF_STR]; if (sq_quote(path, 0, view->vid) >= sizeof(path)) @@ -3690,7 +3712,7 @@ blame_select(struct view *view, struct line *line) if (!commit) return; - if (!strcmp(commit->id, "0000000000000000000000000000000000000000")) + if (!strcmp(commit->id, NULL_ID)) string_ncopy(ref_commit, "HEAD", 4); else string_copy_rev(ref_commit, commit->id); @@ -3724,6 +3746,7 @@ struct status { } new; }; +static char status_onbranch[SIZEOF_STR]; static struct status stage_status; static enum line_type stage_line_type; @@ -3761,7 +3784,7 @@ status_get_diff(struct status *file, char *buf, size_t bufsize) } static bool -status_run(struct view *view, const char cmd[], bool diff, enum line_type type) +status_run(struct view *view, const char cmd[], char status, enum line_type type) { struct status *file = NULL; struct status *unmerged = NULL; @@ -3800,8 +3823,10 @@ status_run(struct view *view, const char cmd[], bool diff, enum line_type type) } /* Parse diff info part. */ - if (!diff) { - file->status = '?'; + if (status) { + file->status = status; + if (status == 'A') + string_copy(file->old.rev, NULL_ID); } else if (!file->status) { if (!status_get_diff(file, buf, sepsize)) @@ -3877,6 +3902,8 @@ error_out: #define STATUS_DIFF_FILES_CMD "git diff-files -z" #define STATUS_LIST_OTHER_CMD \ "git ls-files -z --others --exclude-per-directory=.gitignore" +#define STATUS_LIST_NO_HEAD_CMD \ + "git ls-files -z --cached --exclude-per-directory=.gitignore" #define STATUS_DIFF_INDEX_SHOW_CMD \ "git diff-index --root --patch-with-stat -C -M --cached HEAD -- %s %s 2>/dev/null" @@ -3884,6 +3911,9 @@ error_out: #define STATUS_DIFF_FILES_SHOW_CMD \ "git diff-files --root --patch-with-stat -C -M -- %s %s 2>/dev/null" +#define STATUS_DIFF_NO_HEAD_SHOW_CMD \ + "git diff --no-color --patch-with-stat /dev/null %s 2>/dev/null" + /* First parse staged info using git-diff-index(1), then parse unstaged * info using git-diff-files(1), and finally untracked files using * git-ls-files(1). */ @@ -3892,8 +3922,10 @@ status_open(struct view *view) { struct stat statbuf; char exclude[SIZEOF_STR]; - char cmd[SIZEOF_STR]; + char indexcmd[SIZEOF_STR] = STATUS_DIFF_INDEX_CMD; + char othercmd[SIZEOF_STR] = STATUS_LIST_OTHER_CMD; unsigned long prev_lineno = view->lineno; + char indexstatus = 0; size_t i; for (i = 0; i < view->lines; i++) @@ -3902,35 +3934,66 @@ status_open(struct view *view) view->lines = view->line_alloc = view->line_size = view->lineno = 0; view->line = NULL; - if (!realloc_lines(view, view->line_size + 6)) + if (!realloc_lines(view, view->line_size + 7)) return FALSE; - if (!string_format(exclude, "%s/info/exclude", opt_git_dir)) + add_line_data(view, NULL, LINE_STAT_HEAD); + if (opt_no_head) + string_copy(status_onbranch, "Initial commit"); + else if (!*opt_head) + string_copy(status_onbranch, "Not currently on any branch"); + else if (!string_format(status_onbranch, "On branch %s", opt_head)) return FALSE; - string_copy(cmd, STATUS_LIST_OTHER_CMD); + if (opt_no_head) { + string_copy(indexcmd, STATUS_LIST_NO_HEAD_CMD); + indexstatus = 'A'; + } + + if (!string_format(exclude, "%s/info/exclude", opt_git_dir)) + return FALSE; if (stat(exclude, &statbuf) >= 0) { - size_t cmdsize = strlen(cmd); + size_t cmdsize = strlen(othercmd); - if (!string_format_from(cmd, &cmdsize, " %s", "--exclude-from=") || - sq_quote(cmd, cmdsize, exclude) >= sizeof(cmd)) + if (!string_format_from(othercmd, &cmdsize, " %s", "--exclude-from=") || + sq_quote(othercmd, cmdsize, exclude) >= sizeof(othercmd)) + return FALSE; + + cmdsize = strlen(indexcmd); + if (opt_no_head && + (!string_format_from(indexcmd, &cmdsize, " %s", "--exclude-from=") || + sq_quote(indexcmd, cmdsize, exclude) >= sizeof(indexcmd))) return FALSE; } system("git update-index -q --refresh"); - if (!status_run(view, STATUS_DIFF_INDEX_CMD, TRUE, LINE_STAT_STAGED) || - !status_run(view, STATUS_DIFF_FILES_CMD, TRUE, LINE_STAT_UNSTAGED) || - !status_run(view, cmd, FALSE, LINE_STAT_UNTRACKED)) + if (!status_run(view, indexcmd, indexstatus, LINE_STAT_STAGED) || + !status_run(view, STATUS_DIFF_FILES_CMD, 0, LINE_STAT_UNSTAGED) || + !status_run(view, othercmd, '?', LINE_STAT_UNTRACKED)) return FALSE; /* If all went well restore the previous line number to stay in - * the context. */ + * the context or select a line with something that can be + * updated. */ + if (prev_lineno >= view->lines) + prev_lineno = view->lines - 1; + while (prev_lineno < view->lines && !view->line[prev_lineno].data) + prev_lineno++; + while (prev_lineno > 0 && !view->line[prev_lineno].data) + prev_lineno--; + + /* If the above fails, always skip the "On branch" line. */ if (prev_lineno < view->lines) view->lineno = prev_lineno; else - view->lineno = view->lines - 1; + view->lineno = 1; + + if (view->lineno < view->offset) + view->offset = view->lineno; + else if (view->offset + view->height <= view->lineno) + view->offset = view->lineno - view->height + 1; return TRUE; } @@ -3939,14 +4002,16 @@ static bool status_draw(struct view *view, struct line *line, unsigned int lineno, bool selected) { struct status *status = line->data; - int tilde_attr = get_line_attr(LINE_MAIN_DELIM); wmove(view->win, lineno, 0); if (selected) { wattrset(view->win, get_line_attr(LINE_CURSOR)); wchgat(view->win, -1, 0, LINE_CURSOR, NULL); - tilde_attr = -1; + + } else if (line->type == LINE_STAT_HEAD) { + wattrset(view->win, get_line_attr(LINE_STAT_HEAD)); + wchgat(view->win, -1, 0, LINE_STAT_HEAD, NULL); } else if (!status && line->type != LINE_STAT_NONE) { wattrset(view->win, get_line_attr(LINE_STAT_SECTION)); @@ -3976,11 +4041,15 @@ status_draw(struct view *view, struct line *line, unsigned int lineno, bool sele text = " (no files)"; break; + case LINE_STAT_HEAD: + text = status_onbranch; + break; + default: return FALSE; } - draw_text(view, text, view->width, TRUE, tilde_attr); + draw_text(view, text, view->width, TRUE, selected); return TRUE; } @@ -3991,7 +4060,7 @@ status_draw(struct view *view, struct line *line, unsigned int lineno, bool sele if (view->width < 5) return TRUE; - draw_text(view, status->new.name, view->width - 5, TRUE, tilde_attr); + draw_text(view, status->new.name, view->width - 5, TRUE, selected); return TRUE; } @@ -4027,9 +4096,18 @@ status_enter(struct view *view, struct line *line) switch (line->type) { case LINE_STAT_STAGED: - if (!string_format_from(opt_cmd, &cmdsize, - STATUS_DIFF_INDEX_SHOW_CMD, oldpath, newpath)) - return REQ_QUIT; + if (opt_no_head) { + if (!string_format_from(opt_cmd, &cmdsize, + STATUS_DIFF_NO_HEAD_SHOW_CMD, + newpath)) + return REQ_QUIT; + } else { + if (!string_format_from(opt_cmd, &cmdsize, + STATUS_DIFF_INDEX_SHOW_CMD, + oldpath, newpath)) + return REQ_QUIT; + } + if (status) info = "Staged changes to %s"; else @@ -4050,7 +4128,6 @@ status_enter(struct view *view, struct line *line) if (opt_pipe) return REQ_QUIT; - if (!status) { report("No file to show"); return REQ_NONE; @@ -4060,6 +4137,9 @@ status_enter(struct view *view, struct line *line) info = "Untracked file %s"; break; + case LINE_STAT_HEAD: + return REQ_NONE; + default: die("line type %d not handled in switch", line->type); } @@ -4080,61 +4160,113 @@ status_enter(struct view *view, struct line *line) } -static bool -status_update_file(struct view *view, struct status *status, enum line_type type) +static FILE * +status_update_prepare(enum line_type type) { char cmd[SIZEOF_STR]; - char buf[SIZEOF_STR]; size_t cmdsize = 0; - size_t bufsize = 0; - size_t written = 0; - FILE *pipe; if (opt_cdup[0] && type != LINE_STAT_UNTRACKED && !string_format_from(cmd, &cmdsize, "cd %s;", opt_cdup)) - return FALSE; + return NULL; + + switch (type) { + case LINE_STAT_STAGED: + string_add(cmd, cmdsize, "git update-index -z --index-info"); + break; + + case LINE_STAT_UNSTAGED: + case LINE_STAT_UNTRACKED: + string_add(cmd, cmdsize, "git update-index -z --add --remove --stdin"); + break; + + default: + die("line type %d not handled in switch", type); + } + + return popen(cmd, "w"); +} + +static bool +status_update_write(FILE *pipe, struct status *status, enum line_type type) +{ + char buf[SIZEOF_STR]; + size_t bufsize = 0; + size_t written = 0; switch (type) { case LINE_STAT_STAGED: if (!string_format_from(buf, &bufsize, "%06o %s\t%s%c", - status->old.mode, + status->old.mode, status->old.rev, status->old.name, 0)) return FALSE; - - string_add(cmd, cmdsize, "git update-index -z --index-info"); break; case LINE_STAT_UNSTAGED: case LINE_STAT_UNTRACKED: if (!string_format_from(buf, &bufsize, "%s%c", status->new.name, 0)) return FALSE; - - string_add(cmd, cmdsize, "git update-index -z --add --remove --stdin"); break; default: die("line type %d not handled in switch", type); } - pipe = popen(cmd, "w"); - if (!pipe) - return FALSE; - while (!ferror(pipe) && written < bufsize) { written += fwrite(buf + written, 1, bufsize - written, pipe); } + return written == bufsize; +} + +static bool +status_update_file(struct status *status, enum line_type type) +{ + FILE *pipe = status_update_prepare(type); + bool result; + + if (!pipe) + return FALSE; + + result = status_update_write(pipe, status, type); pclose(pipe); + return result; +} + +static bool +status_update_files(struct view *view, struct line *line) +{ + FILE *pipe = status_update_prepare(line->type); + bool result = TRUE; + struct line *pos = view->line + view->lines; + int files = 0; + int file, done; - if (written != bufsize) + if (!pipe) return FALSE; - return TRUE; + for (pos = line; pos < view->line + view->lines && pos->data; pos++) + files++; + + for (file = 0, done = 0; result && file < files; line++, file++) { + int almost_done = file * 100 / files; + + if (almost_done > done) { + done = almost_done; + string_format(view->ref, "updating file %u of %u (%d%% done)", + file, files, done); + update_view_title(view); + } + result = status_update_write(pipe, line->data, line->type); + } + + pclose(pipe); + return result; } -static void +static bool status_update(struct view *view) { struct line *line = &view->line[view->lineno]; @@ -4142,19 +4274,20 @@ status_update(struct view *view) assert(view->lines); if (!line->data) { - while (++line < view->line + view->lines && line->data) { - if (!status_update_file(view, line->data, line->type)) - report("Failed to update file status"); - } - - if (!line[-1].data) { + /* This should work even for the "On branch" line. */ + if (line < view->line + view->lines && !line[1].data) { report("Nothing to update"); - return; + return FALSE; } - } else if (!status_update_file(view, line->data, line->type)) { + if (!status_update_files(view, line + 1)) + report("Failed to update file status"); + + } else if (!status_update_file(line->data, line->type)) { report("Failed to update file status"); } + + return TRUE; } static enum request @@ -4164,7 +4297,8 @@ status_request(struct view *view, enum request request, struct line *line) switch (request) { case REQ_STATUS_UPDATE: - status_update(view); + if (!status_update(view)) + return REQ_NONE; break; case REQ_STATUS_MERGE: @@ -4236,6 +4370,7 @@ status_select(struct view *view, struct line *line) text = "Press %s to stage %s for addition"; break; + case LINE_STAT_HEAD: case LINE_STAT_NONE: text = "Nothing to update"; break; @@ -4352,7 +4487,7 @@ stage_update_chunk(struct view *view, struct line *line) return FALSE; if (!string_format_from(cmd, &cmdsize, - "git apply --cached %s - && " + "git apply --whitespace=nowarn --cached %s - && " "git update-index -q --unmerged --refresh 2>/dev/null", stage_line_type == LINE_STAT_STAGED ? "-R" : "")) return FALSE; @@ -4407,14 +4542,14 @@ stage_update_chunk(struct view *view, struct line *line) static void stage_update(struct view *view, struct line *line) { - if (stage_line_type != LINE_STAT_UNTRACKED && + if (!opt_no_head && stage_line_type != LINE_STAT_UNTRACKED && (line->type == LINE_DIFF_CHUNK || !stage_status.status)) { if (!stage_update_chunk(view, line)) { report("Failed to apply chunk"); return; } - } else if (!status_update_file(view, &stage_status, stage_line_type)) { + } else if (!status_update_file(&stage_status, stage_line_type)) { report("Failed to update file"); return; } @@ -4688,7 +4823,6 @@ main_draw(struct view *view, struct line *line, unsigned int lineno, bool select enum line_type type; int col = 0; size_t timelen; - int tilde_attr; int space; if (!*commit->author) @@ -4701,19 +4835,17 @@ main_draw(struct view *view, struct line *line, unsigned int lineno, bool select type = LINE_CURSOR; wattrset(view->win, get_line_attr(type)); wchgat(view->win, -1, 0, type, NULL); - tilde_attr = -1; } else { type = LINE_MAIN_COMMIT; wattrset(view->win, get_line_attr(LINE_MAIN_DATE)); - tilde_attr = get_line_attr(LINE_MAIN_DELIM); } if (opt_date) { int n; timelen = strftime(buf, sizeof(buf), DATE_FORMAT, &commit->time); - n = draw_text(view, buf, view->width - col, FALSE, tilde_attr); - draw_text(view, " ", view->width - col - n, FALSE, tilde_attr); + n = draw_text(view, buf, view->width - col, FALSE, selected); + draw_text(view, " ", view->width - col - n, FALSE, selected); col += DATE_COLS; wmove(view->win, lineno, col); @@ -4729,7 +4861,7 @@ main_draw(struct view *view, struct line *line, unsigned int lineno, bool select max_len = view->width - col; if (max_len > AUTHOR_COLS - 1) max_len = AUTHOR_COLS - 1; - draw_text(view, commit->author, max_len, TRUE, tilde_attr); + draw_text(view, commit->author, max_len, TRUE, selected); col += AUTHOR_COLS; if (col >= view->width) return TRUE; @@ -4776,13 +4908,13 @@ main_draw(struct view *view, struct line *line, unsigned int lineno, bool select else wattrset(view->win, get_line_attr(LINE_MAIN_REF)); - col += draw_text(view, "[", view->width - col, TRUE, tilde_attr); + col += draw_text(view, "[", view->width - col, TRUE, selected); col += draw_text(view, commit->refs[i]->name, view->width - col, - TRUE, tilde_attr); - col += draw_text(view, "]", view->width - col, TRUE, tilde_attr); + TRUE, selected); + col += draw_text(view, "]", view->width - col, TRUE, selected); if (type != LINE_CURSOR) wattrset(view->win, A_NORMAL); - col += draw_text(view, " ", view->width - col, TRUE, tilde_attr); + col += draw_text(view, " ", view->width - col, TRUE, selected); if (col >= view->width) return TRUE; } while (commit->refs[i++]->next); @@ -4791,7 +4923,7 @@ main_draw(struct view *view, struct line *line, unsigned int lineno, bool select if (type != LINE_CURSOR) wattrset(view->win, get_line_attr(type)); - draw_text(view, commit->title, view->width - col, TRUE, tilde_attr); + draw_text(view, commit->title, view->width - col, TRUE, selected); return TRUE; } @@ -5406,6 +5538,7 @@ read_ref(char *id, size_t idlen, char *name, size_t namelen) head = !strncmp(opt_head, name, namelen); } else if (!strcmp(name, "HEAD")) { + opt_no_head = FALSE; return OK; } @@ -5625,7 +5758,7 @@ main(int argc, char *argv[]) return 0; /* Require a git repository unless when running in pager mode. */ - if (!opt_git_dir[0]) + if (!opt_git_dir[0] && opt_request != REQ_VIEW_PAGER) die("Not a git repository"); if (*opt_encoding && strcasecmp(opt_encoding, "UTF-8"))