diff options
| author | Jeff King <peff@peff.net> | 2023-11-09 02:13:12 -0500 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2023-11-09 19:07:53 +0900 |
| commit | 92de4c5d56d084325997ca057701b65a9e79276a (patch) | |
| tree | 9e100c2f52f9437a2261e551ec58fdc2267781b2 /builtin/commit.c | |
| parent | midx: check consistency of fanout table (diff) | |
| download | git-92de4c5d56d084325997ca057701b65a9e79276a.tar.gz git-92de4c5d56d084325997ca057701b65a9e79276a.zip | |
commit-graph: drop redundant call to "lite" verification
The idea of verify_commit_graph_lite() is to have cheap verification
checks both for everyday use of the graph files (to avoid out of bounds
reads, etc) as well as for doing a full check via "commit-graph verify"
(which will also check the hash, etc).
But the expensive verification checks operate on a commit_graph struct,
which we get by using the normal everyday-reader code! So any problem
we'd find by calling it would have been found before we even got to the
verify_one_commit_graph() function.
Removing it simplifies the code a bit, but also frees us up to move the
"lite" verification steps around within that everyday-reader code.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/commit.c')
0 files changed, 0 insertions, 0 deletions
