Jeff King <peff@xxxxxxxx> writes: > > I didn't explore any of those here, and I don't plan to look into them > anytime soon. I'm just documenting my findings for later. > > Anyway, here are the patches. > > [1/3]: is_promisor_object(): free tree buffer after parsing > [2/3]: lookup_unknown_object(): take a repository argument > [3/3]: revision: avoid parsing with --exclude-promisor-objects > > builtin/fsck.c | 2 +- > builtin/pack-objects.c | 2 +- > http-push.c | 2 +- > object.c | 7 +++---- > object.h | 2 +- > packfile.c | 1 + > refs.c | 2 +- > revision.c | 2 +- > t/helper/test-example-decorate.c | 6 +++--- > t/perf/p5600-partial-clone.sh | 12 ++++++++++++ > upload-pack.c | 2 +- > walker.c | 2 +- > 12 files changed, 27 insertions(+), 15 deletions(-) > > -Peff I took look on this series and tested as well, together with the fix for the "unpacking and deleting" promisor objects situation. It looks good to me. -- Thanks Rafael