Browse Source

Fix "git log -i --grep"

This has been broken in v1.6.0 due to the reorganization of
the revision option parsing code. The "-i" is completely
ignored, but works fine in "git log --grep -i".

What happens is that the code for "-i" looks for
revs->grep_filter; if it is NULL, we do nothing, since there
are no grep filters. But that is obviously not correct,
since we want it to influence the later --grep option. Doing
it the other way around works, since "-i" just impacts the
existing grep_filter option.

Instead, we now always initialize the grep_filter member and
just fill in options and patterns as we get them. This means
that we can no longer check grep_filter for NULL, but
instead must check the pattern list to see if we have any
actual patterns.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
maint
Jeff King 17 years ago committed by Junio C Hamano
parent
commit
0843acfd2c
  1. 3
      builtin-rev-list.c
  2. 34
      revision.c
  3. 3
      revision.h
  4. 22
      t/t4202-log.sh

3
builtin-rev-list.c

@ -645,7 +645,8 @@ int cmd_rev_list(int argc, const char **argv, const char *prefix) @@ -645,7 +645,8 @@ int cmd_rev_list(int argc, const char **argv, const char *prefix)
revs.diff)
usage(rev_list_usage);

save_commit_buffer = revs.verbose_header || revs.grep_filter;
save_commit_buffer = revs.verbose_header ||
revs.grep_filter.pattern_list;
if (bisect_list)
revs.limited = 1;


34
revision.c

@ -782,6 +782,10 @@ void init_revisions(struct rev_info *revs, const char *prefix) @@ -782,6 +782,10 @@ void init_revisions(struct rev_info *revs, const char *prefix)

revs->commit_format = CMIT_FMT_DEFAULT;

revs->grep_filter.status_only = 1;
revs->grep_filter.pattern_tail = &(revs->grep_filter.pattern_list);
revs->grep_filter.regflags = REG_NEWLINE;

diff_setup(&revs->diffopt);
if (prefix && !revs->diffopt.prefix) {
revs->diffopt.prefix = prefix;
@ -946,15 +950,7 @@ void read_revisions_from_stdin(struct rev_info *revs) @@ -946,15 +950,7 @@ void read_revisions_from_stdin(struct rev_info *revs)

static void add_grep(struct rev_info *revs, const char *ptn, enum grep_pat_token what)
{
if (!revs->grep_filter) {
struct grep_opt *opt = xcalloc(1, sizeof(*opt));
opt->status_only = 1;
opt->pattern_tail = &(opt->pattern_list);
opt->regflags = REG_NEWLINE;
revs->grep_filter = opt;
}
append_grep_pattern(revs->grep_filter, ptn,
"command line", 0, what);
append_grep_pattern(&revs->grep_filter, ptn, "command line", 0, what);
}

static void add_header_grep(struct rev_info *revs, const char *field, const char *pattern)
@ -1164,17 +1160,13 @@ static int handle_revision_opt(struct rev_info *revs, int argc, const char **arg @@ -1164,17 +1160,13 @@ static int handle_revision_opt(struct rev_info *revs, int argc, const char **arg
} else if (!prefixcmp(arg, "--grep=")) {
add_message_grep(revs, arg+7);
} else if (!strcmp(arg, "--extended-regexp") || !strcmp(arg, "-E")) {
if (revs->grep_filter)
revs->grep_filter->regflags |= REG_EXTENDED;
revs->grep_filter.regflags |= REG_EXTENDED;
} else if (!strcmp(arg, "--regexp-ignore-case") || !strcmp(arg, "-i")) {
if (revs->grep_filter)
revs->grep_filter->regflags |= REG_ICASE;
revs->grep_filter.regflags |= REG_ICASE;
} else if (!strcmp(arg, "--fixed-strings") || !strcmp(arg, "-F")) {
if (revs->grep_filter)
revs->grep_filter->fixed = 1;
revs->grep_filter.fixed = 1;
} else if (!strcmp(arg, "--all-match")) {
if (revs->grep_filter)
revs->grep_filter->all_match = 1;
revs->grep_filter.all_match = 1;
} else if (!prefixcmp(arg, "--encoding=")) {
arg += 11;
if (strcmp(arg, "none"))
@ -1349,9 +1341,7 @@ int setup_revisions(int argc, const char **argv, struct rev_info *revs, const ch @@ -1349,9 +1341,7 @@ int setup_revisions(int argc, const char **argv, struct rev_info *revs, const ch
if (diff_setup_done(&revs->diffopt) < 0)
die("diff_setup_done failed");

if (revs->grep_filter) {
compile_grep_patterns(revs->grep_filter);
}
compile_grep_patterns(&revs->grep_filter);

if (revs->reverse && revs->reflog_info)
die("cannot combine --reverse with --walk-reflogs");
@ -1492,9 +1482,9 @@ static int rewrite_parents(struct rev_info *revs, struct commit *commit) @@ -1492,9 +1482,9 @@ static int rewrite_parents(struct rev_info *revs, struct commit *commit)

static int commit_match(struct commit *commit, struct rev_info *opt)
{
if (!opt->grep_filter)
if (!opt->grep_filter.pattern_list)
return 1;
return grep_buffer(opt->grep_filter,
return grep_buffer(&opt->grep_filter,
NULL, /* we say nothing, not even filename */
commit->buffer, strlen(commit->buffer));
}

3
revision.h

@ -2,6 +2,7 @@ @@ -2,6 +2,7 @@
#define REVISION_H

#include "parse-options.h"
#include "grep.h"

#define SEEN (1u<<0)
#define UNINTERESTING (1u<<1)
@ -92,7 +93,7 @@ struct rev_info { @@ -92,7 +93,7 @@ struct rev_info {
int show_log_size;

/* Filter by commit log message */
struct grep_opt *grep_filter;
struct grep_opt grep_filter;

/* Display history graph */
struct git_graph *graph;

22
t/t4202-log.sh

@ -69,7 +69,29 @@ test_expect_success 'diff-filter=D' ' @@ -69,7 +69,29 @@ test_expect_success 'diff-filter=D' '

'

test_expect_success 'setup case sensitivity tests' '
echo case >one &&
test_tick &&
git commit -a -m Second
'

test_expect_success 'log --grep' '
echo second >expect &&
git log -1 --pretty="tformat:%s" --grep=sec >actual &&
test_cmp expect actual
'

test_expect_success 'log -i --grep' '
echo Second >expect &&
git log -1 --pretty="tformat:%s" -i --grep=sec >actual &&
test_cmp expect actual
'

test_expect_success 'log --grep -i' '
echo Second >expect &&
git log -1 --pretty="tformat:%s" --grep=sec -i >actual &&
test_cmp expect actual
'

test_done


Loading…
Cancel
Save