[PATCH v5 14/17] commit-graph: drop unnecessary `graph_read_bloom_data_context`
- From
Taylor Blau <me@ttaylorr.com>
- Date
- Jan 16, 2024, 22:09 UTC
- Message-ID
- <526beb9766a25dc97b9a4913fc02701908c0612e.1705442923.git.me@ttaylorr.com>
- In-Reply-To
- <cover.1705442923.git.me@ttaylorr.com>
The `graph_read_bloom_data_context` struct was introduced in an earlier commit in order to pass pointers to the commit-graph and changed-path Bloom filter version when reading the BDAT chunk.
The previous commit no longer writes through the changed_paths_version pointer, making the surrounding context structure unnecessary. Drop it and pass a pointer to the commit-graph directly when reading the BDAT chunk.
Noticed-by: Jonathan Tan <jonathantanmy@google.com> Signed-off-by: Taylor Blau <me@ttaylorr.com> --- commit-graph.c | 14 ++------------ 1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/commit-graph.c b/commit-graph.c index a2063d5f91..a02556716d 100644 --- a/commit-graph.c +++ b/commit-graph.c @@ -340,16 +340,10 @@ static int graph_read_bloom_index(const unsigned char *chunk_start, return 0; } -struct graph_read_bloom_data_context { - struct commit_graph *g; - int *commit_graph_changed_paths_version; -}; - static int graph_read_bloom_data(const unsigned char *chunk_start, size_t chunk_size, void *data) { - struct graph_read_bloom_data_context *c = data; - struct commit_graph *g = c->g; + struct commit_graph *g = data; uint32_t hash_version; if (chunk_size < BLOOMDATA_CHUNK_HEADER_SIZE) { @@ -463,14 +457,10 @@ struct commit_graph *parse_commit_graph(struct repo_settings *s, } if (s->commit_graph_changed_paths_version) { - struct graph_read_bloom_data_context context = { - .g = graph, - .commit_graph_changed_paths_version = &s->commit_graph_changed_paths_version - }; read_chunk(cf, GRAPH_CHUNKID_BLOOMINDEXES, graph_read_bloom_index, graph); read_chunk(cf, GRAPH_CHUNKID_BLOOMDATA, - graph_read_bloom_data, &context); + graph_read_bloom_data, graph); } if (graph->chunk_bloom_indexes && graph->chunk_bloom_data) {
-- 2.43.0.334.gd4dbce1db5.dirty