]> Kevux Git Server - rit/commit
revision: drop explicit check for commit graph
authorPatrick Steinhardt <ps@pks.im>
Thu, 4 Sep 2025 12:49:56 +0000 (14:49 +0200)
committerJunio C Hamano <gitster@pobox.com>
Thu, 4 Sep 2025 23:16:21 +0000 (16:16 -0700)
commit307e30792b3fb256f58816c4ca67d647eb76c788
treeb776bf4d7f3210bd57298820e7897e8fb0de6779
parentd5a6f505e65444bafeae6d86568c589c3be8797b
revision: drop explicit check for commit graph

When filtering down revisions by paths we know to use bloom filters from
the commit graph, if we have any. The entry point for this is in
`check_maybe_different_in_bloom_filter()`, where we first verify that:

  - We do have a commit graph.

  - That the commit is contained therein by checking that we have a
    proper generation number.

  - And that the graph contains a bloom filter.

The first check is somewhat redundant though: if we don't have a commit
graph, then the second check would already tell us that we don't have a
generation number for the specific commit.

In theory this could be seen as a performance optimization to
short-circuit for scenarios where there is no commit graph. But in
practice this shouldn't matter: if there is no commit graph, then the
commit graph data slab would also be unpopulated and thus a lookup of
the commit should happen in constant time.

Drop the unnecessary check.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
revision.c