Re: [PATCH v3 2/2] builtin/pack-objects.c: introduce `pack.recentObjectsHook`
- From
Taylor Blau <me@ttaylorr.com>
- Date
- May 15, 2023, 20:15 UTC
- Message-ID
- <ZGKS3gB5B7fbetFb@nand.local>
- In-Reply-To
- <20230512045853.GA2479240@coredump.intra.peff.net>
On Fri, May 12, 2023 at 12:58:53AM -0400, Jeff King wrote:
Show 6 quoted lines
> I haven't looked closely at this whole patch yet (and I especially want > to look at the new tests since this approach covers more cases), but I > did notice that this version of the function still has the "we don't > reap the child on parse failure" problem I described in: > > https://lore.kernel.org/git/20230505221921.GE3321533@coredump.intra.peff.net/
Hmmph. I could have sworn that I remember including that feedback in the new round, but I must have dropped it on the floor somewhere.
Thanks for pointing it out, I'm 99% sure that it'll be in the next round ;-).
Thanks, Taylor