]> git.ipfire.org Git - thirdparty/git.git/commit - revision.c
revision: complicated pathspecs disable filters
authorDerrick Stolee <dstolee@microsoft.com>
Thu, 16 Apr 2020 20:14:02 +0000 (20:14 +0000)
committerJunio C Hamano <gitster@pobox.com>
Thu, 16 Apr 2020 22:38:02 +0000 (15:38 -0700)
commit8918e379aac11b1e2ed7bb5be8433516c4ff5e8f
treec1b8abdd97dd885b238876172bf51deba4ff245a
parentcaf388caa101be90b7ec43d7f78ca4e935fc0150
revision: complicated pathspecs disable filters

The changed-path Bloom filters work only when we can compute an
explicit Bloom filter key in advance. When a pathspec is given
that allows case-insensitive checks or wildcard matching, we
must disable the Bloom filter performance checks.

By checking the pathspec in prepare_to_use_bloom_filters(), we
avoid setting up the Bloom filter data and thus revert to the
usual logic.

Before this change, the following tests would fail*:

t6004-rev-list-path-optim.sh (Tests 6-7)
t6130-pathspec-noglob.sh (Tests 3-6)
t6131-pathspec-icase.sh (Tests 3-5)

*These tests would fail when using GIT_TEST_COMMIT_GRAPH and
GIT_TEST_COMMIT_GRAPH_BLOOM_FILTERS except that the latter
environment variable was not set up correctly to write the changed-
path Bloom filters in the test suite. That will be fixed in the
next change.

Helped-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
revision.c