From: Patrick Steinhardt Date: Wed, 24 Jun 2026 12:19:17 +0000 (+0200) Subject: treewide: convert users of `whence` to the new source field X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=4f48d2a241037fbc6bd4bd968ce77681d5b0e610;p=thirdparty%2Fgit.git treewide: convert users of `whence` to the new source field The `whence` field has become redundant now that callers can learn about the exact source an object has been looked up from via the `struct object_info_source::source` field. Adapt callers to use the new field. Note that all callsites already set up the `info.sourcep` request pointer, so the conversion is rather straight-forward. Signed-off-by: Patrick Steinhardt Signed-off-by: Junio C Hamano --- diff --git a/builtin/cat-file.c b/builtin/cat-file.c index adc626ce30..1b96150e5b 100644 --- a/builtin/cat-file.c +++ b/builtin/cat-file.c @@ -834,7 +834,7 @@ static int batch_one_object_oi(const struct object_id *oid, void *_payload) { struct for_each_object_payload *payload = _payload; - if (oi && oi->whence == OI_PACKED) + if (oi && oi->sourcep->source->type == ODB_SOURCE_PACKED) return payload->callback(oid, oi->sourcep->u.packed.pack, oi->sourcep->u.packed.offset, payload->payload); diff --git a/builtin/index-pack.c b/builtin/index-pack.c index 77af26db8f..1b03b07e5e 100644 --- a/builtin/index-pack.c +++ b/builtin/index-pack.c @@ -1833,7 +1833,8 @@ static void repack_local_links(void) if (odb_read_object_info_extended(the_repository->objects, oid, &info, 0)) /* Missing; assume it is a promisor object */ continue; - if (info.whence == OI_PACKED && info_source.u.packed.pack->pack_promisor) + if (info_source.source->type == ODB_SOURCE_PACKED && + info_source.u.packed.pack->pack_promisor) continue; if (!cmd.args.nr) { diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c index 9deb37e9e8..d0fdfad750 100644 --- a/builtin/pack-objects.c +++ b/builtin/pack-objects.c @@ -5010,7 +5010,7 @@ static int is_not_in_promisor_pack_obj(struct object *obj, void *data UNUSED) if (odb_read_object_info_extended(the_repository->objects, &obj->oid, &info, 0)) BUG("should_include_obj should only be called on existing objects"); - return info.whence != OI_PACKED || !info_source.u.packed.pack->pack_promisor; + return info_source.source->type != ODB_SOURCE_PACKED || !info_source.u.packed.pack->pack_promisor; } static int is_not_in_promisor_pack(struct commit *commit, void *data) { diff --git a/reachable.c b/reachable.c index 2fc5b82d62..123a658944 100644 --- a/reachable.c +++ b/reachable.c @@ -234,7 +234,7 @@ static int add_recent_object(const struct object_id *oid, add_pending_object(data->revs, obj, ""); if (data->cb) { - if (oi->whence == OI_PACKED) + if (oi->sourcep->source->type == ODB_SOURCE_PACKED) data->cb(obj, oi->sourcep->u.packed.pack, oi->sourcep->u.packed.offset, *oi->mtimep); else