aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPatrick Steinhardt <ps@pks.im>2025-07-15 13:29:24 +0200
committerJunio C Hamano <gitster@pobox.com>2025-07-15 12:07:30 -0700
commitec865d94d4615c00fbf9ac50f4274b1d3fbf73a6 (patch)
tree81c186dea2a578e987ba03eab2f232a140739fbb
parentpackfile: stop using linked MIDX list in `get_all_packs()` (diff)
downloadgit-ec865d94d4615c00fbf9ac50f4274b1d3fbf73a6.tar.gz
git-ec865d94d4615c00fbf9ac50f4274b1d3fbf73a6.zip
midx: remove now-unused linked list of multi-pack indices
In the preceding commits we have migrated all users of the linked list of multi-pack indices to instead use those stored in the object database sources. Remove those now-unused pointers. Signed-off-by: Patrick Steinhardt <ps@pks.im> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--midx.c18
-rw-r--r--midx.h2
-rw-r--r--odb.h7
-rw-r--r--packfile.c1
4 files changed, 2 insertions, 26 deletions
diff --git a/midx.c b/midx.c
index 472d6bf17a..7d407682e6 100644
--- a/midx.c
+++ b/midx.c
@@ -726,7 +726,6 @@ int midx_preferred_pack(struct multi_pack_index *m, uint32_t *pack_int_id)
int prepare_multi_pack_index_one(struct odb_source *source, int local)
{
struct repository *r = source->odb->repo;
- struct multi_pack_index *m;
prepare_repo_settings(r);
if (!r->settings.core_multi_pack_index)
@@ -735,21 +734,9 @@ int prepare_multi_pack_index_one(struct odb_source *source, int local)
if (source->midx)
return 1;
- m = load_multi_pack_index(r, source->path, local);
- if (m) {
- struct multi_pack_index *mp = r->objects->multi_pack_index;
- if (mp) {
- m->next = mp->next;
- mp->next = m;
- } else {
- r->objects->multi_pack_index = m;
- }
- source->midx = m;
+ source->midx = load_multi_pack_index(r, source->path, local);
- return 1;
- }
-
- return 0;
+ return !!source->midx;
}
int midx_checksum_valid(struct multi_pack_index *m)
@@ -842,7 +829,6 @@ void clear_midx_file(struct repository *r)
close_midx(source->midx);
source->midx = NULL;
}
- r->objects->multi_pack_index = NULL;
}
if (remove_path(midx.buf))
diff --git a/midx.h b/midx.h
index 639a6f50e4..076382de8a 100644
--- a/midx.h
+++ b/midx.h
@@ -35,8 +35,6 @@ struct odb_source;
"GIT_TEST_MULTI_PACK_INDEX_WRITE_INCREMENTAL"
struct multi_pack_index {
- struct multi_pack_index *next;
-
const unsigned char *data;
size_t data_len;
diff --git a/odb.h b/odb.h
index f09dba1fe1..09177bf430 100644
--- a/odb.h
+++ b/odb.h
@@ -126,13 +126,6 @@ struct object_database {
/*
* private data
*
- * should only be accessed directly by packfile.c and midx.c
- */
- struct multi_pack_index *multi_pack_index;
-
- /*
- * private data
- *
* should only be accessed directly by packfile.c
*/
diff --git a/packfile.c b/packfile.c
index ff33692f4b..5d73932f50 100644
--- a/packfile.c
+++ b/packfile.c
@@ -375,7 +375,6 @@ void close_object_store(struct object_database *o)
close_midx(source->midx);
source->midx = NULL;
}
- o->multi_pack_index = NULL;
close_commit_graph(o);
}