From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS53758 23.128.96.0/24 X-Spam-Status: No, score=-4.0 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI, SPF_HELO_PASS,SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by dcvr.yhbt.net (Postfix) with ESMTP id DE73D1F8C6 for ; Sun, 5 Sep 2021 12:05:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237888AbhIEMGa (ORCPT ); Sun, 5 Sep 2021 08:06:30 -0400 Received: from h2.fbrelay.privateemail.com ([131.153.2.43]:35617 "EHLO h2.fbrelay.privateemail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234382AbhIEMGa (ORCPT ); Sun, 5 Sep 2021 08:06:30 -0400 Received: from MTA-08-3.privateemail.com (mta-08-1.privateemail.com [68.65.122.28]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by h1.fbrelay.privateemail.com (Postfix) with ESMTPS id 34B538021E for ; Sun, 5 Sep 2021 08:05:26 -0400 (EDT) Received: from mta-08.privateemail.com (localhost [127.0.0.1]) by mta-08.privateemail.com (Postfix) with ESMTP id 6E1D018001BE; Sun, 5 Sep 2021 08:05:24 -0400 (EDT) Received: from hal-station.. (unknown [10.20.151.242]) by mta-08.privateemail.com (Postfix) with ESMTPA id BB47D18000BE; Sun, 5 Sep 2021 08:05:23 -0400 (EDT) From: Hamza Mahfooz To: git@vger.kernel.org Cc: Junio C Hamano , Hamza Mahfooz Subject: [PATCH] pretty: colorize pattern matches in commit messages Date: Sun, 5 Sep 2021 08:05:06 -0400 Message-Id: <20210905120506.635130-1-someguy@effective-light.com> X-Mailer: git-send-email 2.33.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Virus-Scanned: ClamAV using ClamSMTP Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org The "git log" command limits its output to the commits that contain strings matched by a pattern when the "--grep=" option is used, but unlike output from "git grep -e ", the matches are not highlighted, making them harder to spot. Teach the pretty-printer code to highlight matches from the "--grep=", "--author=" and "--committer=" options (to view the last one, you may have to ask for --pretty=fuller). Signed-off-by: Hamza Mahfooz --- v2: make the commit message whole (add the missing ingredients), rename append_matched_line() to append_line_with_color(), use colors[GREP_COLOR_MATCH_SELECTED] instead of colors[GREP_COLOR_MATCH_CONTEXT], allow the background color to be customized, don't copy strings to a buffer when not coloring in append_line_with_color(), rename next_match() to grep_next_match(), repurpose grep_next_match()/match_one_pattern() for use in append_line_with_color() (allowing us to remove duplicated matching code in append_line_with_color()), document how to customize the feature and modify some of the tests to fit the feature better. --- Documentation/git-log.txt | 8 +++ grep.c | 41 +++++++++------ grep.h | 3 ++ pretty.c | 108 +++++++++++++++++++++++++++++++++----- t/t4202-log.sh | 42 +++++++++++++++ 5 files changed, 175 insertions(+), 27 deletions(-) diff --git a/Documentation/git-log.txt b/Documentation/git-log.txt index 0498e7bacb..8d08785863 100644 --- a/Documentation/git-log.txt +++ b/Documentation/git-log.txt @@ -232,6 +232,14 @@ notes.displayRef:: or `GIT_NOTES_REF`, to read notes from when showing commit messages with the `log` family of commands. See linkgit:git-notes[1]. + +color.grep.selected:: + Determines the non matching text (background) color of selected lines, + when `--grep`, `--author` or `--committer` are used. + +color.grep.matchSelected:: + Determines the matching text (foreground) color of selected lines, when + `--grep`, `--author` or `--committer` are used. + May be an unabbreviated ref name or a glob and may be specified multiple times. A warning will be issued for refs that do not exist, diff --git a/grep.c b/grep.c index 424a39591b..6b036ee18a 100644 --- a/grep.c +++ b/grep.c @@ -956,18 +956,23 @@ static int match_one_pattern(struct grep_pat *p, char *bol, char *eol, const char *start = bol; if ((p->token != GREP_PATTERN) && - ((p->token == GREP_PATTERN_HEAD) != (ctx == GREP_CONTEXT_HEAD))) + ((p->token == GREP_PATTERN_HEAD) != (ctx == GREP_CONTEXT_HEAD)) && + ((p->token == GREP_PATTERN_BODY) != (ctx == GREP_CONTEXT_BODY))) return 0; if (p->token == GREP_PATTERN_HEAD) { - const char *field; - size_t len; - assert(p->field < ARRAY_SIZE(header_field)); - field = header_field[p->field].field; - len = header_field[p->field].len; - if (strncmp(bol, field, len)) - return 0; - bol += len; + if (!(eflags & REG_NOTBOL)) { + const char *field; + size_t len; + + assert(p->field < ARRAY_SIZE(header_field)); + field = header_field[p->field].field; + len = header_field[p->field].len; + if (strncmp(bol, field, len)) + return 0; + bol += len; + } + switch (p->field) { case GREP_HEADER_AUTHOR: case GREP_HEADER_COMMITTER: @@ -1159,21 +1164,26 @@ static int match_next_pattern(struct grep_pat *p, char *bol, char *eol, return 1; } -static int next_match(struct grep_opt *opt, char *bol, char *eol, - enum grep_context ctx, regmatch_t *pmatch, int eflags) +int grep_next_match(struct grep_opt *opt, char *bol, char *eol, + enum grep_context ctx, regmatch_t *pmatch, + enum grep_header_field field, int eflags) { struct grep_pat *p; int hit = 0; pmatch->rm_so = pmatch->rm_eo = -1; if (bol < eol) { - for (p = opt->pattern_list; p; p = p->next) { + for (p = ((ctx == GREP_CONTEXT_HEAD) + ? opt->header_list : opt->pattern_list); + p; p = p->next) { switch (p->token) { case GREP_PATTERN: /* atom */ case GREP_PATTERN_HEAD: case GREP_PATTERN_BODY: - hit |= match_next_pattern(p, bol, eol, ctx, - pmatch, eflags); + if ((field == GREP_HEADER_FIELD_MAX) || (p->field == field)) + hit |= match_next_pattern(p, bol, eol, + ctx, pmatch, + eflags); break; default: break; @@ -1262,7 +1272,8 @@ static void show_line(struct grep_opt *opt, char *bol, char *eol, line_color = opt->colors[GREP_COLOR_FUNCTION]; } *eol = '\0'; - while (next_match(opt, bol, eol, ctx, &match, eflags)) { + while (grep_next_match(opt, bol, eol, ctx, &match, + GREP_HEADER_FIELD_MAX, eflags)) { if (match.rm_so == match.rm_eo) break; diff --git a/grep.h b/grep.h index 72f82b1e30..d2943e29ea 100644 --- a/grep.h +++ b/grep.h @@ -177,6 +177,9 @@ void append_header_grep_pattern(struct grep_opt *, enum grep_header_field, const void compile_grep_patterns(struct grep_opt *opt); void free_grep_patterns(struct grep_opt *opt); int grep_buffer(struct grep_opt *opt, char *buf, unsigned long size); +int grep_next_match(struct grep_opt *opt, char *bol, char *eol, + enum grep_context ctx, regmatch_t *pmatch, + enum grep_header_field field, int eflags); struct grep_source { char *name; diff --git a/pretty.c b/pretty.c index 9631529c10..daf446ee53 100644 --- a/pretty.c +++ b/pretty.c @@ -431,15 +431,71 @@ const char *show_ident_date(const struct ident_split *ident, return show_date(date, tz, mode); } +static void append_line_with_color(struct strbuf *sb, struct grep_opt *opt, + const char *line, size_t linelen, + int color, enum grep_context ctx, + enum grep_header_field field) +{ + char *buf, *eol; + const char *line_color, *match_color; + regmatch_t match; + struct strbuf tmp_sb; + int eflags = 0; + + if (!opt || !want_color(color) || opt->invert) { + strbuf_add(sb, line, linelen); + return; + } + + if (ctx == GREP_CONTEXT_HEAD) + eflags = REG_NOTBOL; + + strbuf_init(&tmp_sb, linelen + 1); + strbuf_add(&tmp_sb, line, linelen); + + buf = tmp_sb.buf; + eol = buf + linelen; + line_color = opt->colors[GREP_COLOR_SELECTED]; + match_color = opt->colors[GREP_COLOR_MATCH_SELECTED]; + + while (grep_next_match(opt, buf, eol, ctx, &match, field, eflags)) { + if (match.rm_so == match.rm_eo) + break; + + strbuf_grow(sb, strlen(line_color) + strlen(match_color) + + strlen(GIT_COLOR_RESET)); + strbuf_add(sb, line_color, strlen(line_color)); + strbuf_add(sb, buf, match.rm_so); + strbuf_add(sb, match_color, strlen(match_color)); + strbuf_add(sb, buf + match.rm_so, + match.rm_eo - match.rm_so); + strbuf_add(sb, GIT_COLOR_RESET, + strlen(GIT_COLOR_RESET)); + buf += match.rm_eo; + eflags = REG_NOTBOL; + } + + if (buf != line) { + strbuf_grow(sb, strlen(line_color)); + strbuf_add(sb, line_color, strlen(line_color)); + } + + strbuf_add(sb, buf, eol - buf); + strbuf_release(&tmp_sb); +} + void pp_user_info(struct pretty_print_context *pp, const char *what, struct strbuf *sb, const char *line, const char *encoding) { + struct strbuf id; struct ident_split ident; char *line_end; const char *mailbuf, *namebuf; size_t namelen, maillen; int max_length = 78; /* per rfc2822 */ + enum grep_header_field field = GREP_HEADER_FIELD_MAX; + struct grep_opt *opt = pp->rev ? &pp->rev->grep_filter : NULL; if (pp->fmt == CMIT_FMT_ONELINE) return; @@ -496,9 +552,22 @@ void pp_user_info(struct pretty_print_context *pp, strbuf_addch(sb, '\n'); strbuf_addf(sb, " <%.*s>\n", (int)maillen, mailbuf); } else { - strbuf_addf(sb, "%s: %.*s%.*s <%.*s>\n", what, - (pp->fmt == CMIT_FMT_FULLER) ? 4 : 0, " ", - (int)namelen, namebuf, (int)maillen, mailbuf); + strbuf_init(&id, namelen + maillen + 4); + + if (!strcmp(what, "Author")) + field = GREP_HEADER_AUTHOR; + else if (!strcmp(what, "Commit")) + field = GREP_HEADER_COMMITTER; + + strbuf_addf(sb, "%s: %.*s", what, + (pp->fmt == CMIT_FMT_FULLER) ? 4 : 0, " "); + strbuf_addf(&id, "%.*s <%.*s>", (int)namelen, namebuf, + (int)maillen, mailbuf); + + append_line_with_color(sb, opt, id.buf, id.len, pp->color, + GREP_CONTEXT_HEAD, field); + strbuf_addch(sb, '\n'); + strbuf_release(&id); } switch (pp->fmt) { @@ -1855,6 +1924,7 @@ static void pp_header(struct pretty_print_context *pp, } } + void pp_title_line(struct pretty_print_context *pp, const char **msg_p, struct strbuf *sb, @@ -1935,8 +2005,9 @@ static int pp_utf8_width(const char *start, const char *end) return width; } -static void strbuf_add_tabexpand(struct strbuf *sb, int tabwidth, - const char *line, int linelen) +static void strbuf_add_tabexpand(struct strbuf *sb, struct grep_opt *opt, + int color, int tabwidth, const char *line, + int linelen) { const char *tab; @@ -1953,7 +2024,9 @@ static void strbuf_add_tabexpand(struct strbuf *sb, int tabwidth, break; /* Output the data .. */ - strbuf_add(sb, line, tab - line); + append_line_with_color(sb, opt, line, tab - line, color, + GREP_CONTEXT_BODY, + GREP_HEADER_FIELD_MAX); /* .. and the de-tabified tab */ strbuf_addchars(sb, ' ', tabwidth - (width % tabwidth)); @@ -1968,7 +2041,8 @@ static void strbuf_add_tabexpand(struct strbuf *sb, int tabwidth, * worrying about width - there's nothing more to * align. */ - strbuf_add(sb, line, linelen); + append_line_with_color(sb, opt, line, linelen, color, GREP_CONTEXT_BODY, + GREP_HEADER_FIELD_MAX); } /* @@ -1980,11 +2054,16 @@ static void pp_handle_indent(struct pretty_print_context *pp, struct strbuf *sb, int indent, const char *line, int linelen) { + struct grep_opt *opt = pp->rev ? &pp->rev->grep_filter : NULL; + strbuf_addchars(sb, ' ', indent); if (pp->expand_tabs_in_log) - strbuf_add_tabexpand(sb, pp->expand_tabs_in_log, line, linelen); + strbuf_add_tabexpand(sb, opt, pp->color, pp->expand_tabs_in_log, + line, linelen); else - strbuf_add(sb, line, linelen); + append_line_with_color(sb, opt, line, linelen, pp->color, + GREP_CONTEXT_BODY, + GREP_HEADER_FIELD_MAX); } static int is_mboxrd_from(const char *line, int len) @@ -2002,7 +2081,9 @@ void pp_remainder(struct pretty_print_context *pp, struct strbuf *sb, int indent) { + struct grep_opt *opt = pp->rev ? &pp->rev->grep_filter : NULL; int first = 1; + for (;;) { const char *line = *msg_p; int linelen = get_one_line(line); @@ -2023,14 +2104,17 @@ void pp_remainder(struct pretty_print_context *pp, if (indent) pp_handle_indent(pp, sb, indent, line, linelen); else if (pp->expand_tabs_in_log) - strbuf_add_tabexpand(sb, pp->expand_tabs_in_log, - line, linelen); + strbuf_add_tabexpand(sb, opt, pp->color, + pp->expand_tabs_in_log, line, + linelen); else { if (pp->fmt == CMIT_FMT_MBOXRD && is_mboxrd_from(line, linelen)) strbuf_addch(sb, '>'); - strbuf_add(sb, line, linelen); + append_line_with_color(sb, opt, line, linelen, + pp->color, GREP_CONTEXT_BODY, + GREP_HEADER_FIELD_MAX); } strbuf_addch(sb, '\n'); } diff --git a/t/t4202-log.sh b/t/t4202-log.sh index 9dfead936b..72138fa7a0 100755 --- a/t/t4202-log.sh +++ b/t/t4202-log.sh @@ -449,6 +449,48 @@ test_expect_success !FAIL_PREREQS 'log with various grep.patternType configurati ) ' +cat > expect << EOF +Author: A U Thor +Author: A U Thor +Author: A U Thor +Author: A U Thor +Author: A U Thor +Author: A U Thor +Author: A U Thor +EOF + +test_expect_success 'log --author' ' + git log --color=always --author="A U" >log && + grep Author log >actual && + test_cmp expect actual +' + +cat > expect << EOF +Commit: C O Mitter +Commit: C O Mitter +Commit: C O Mitter +Commit: C O Mitter +Commit: C O Mitter +Commit: C O Mitter +Commit: C O Mitter +EOF + +test_expect_success 'log --committer' ' + git log --color=always --pretty=fuller --committer="Mitter" >log && + grep "Commit:" log >actual && + test_cmp expect actual +' + +cat > expect << EOF + second +EOF + +test_expect_success 'log --grep color' ' + git log --color=always --grep=sec >log && + grep sec log >actual && + test_cmp expect actual +' + cat > expect <