From: Jeff King Date: Mon, 03 Jul 2023 06:29:25 GMT Subject: Re: [PATCH v4 00/16] refs: implement jump lists for packed backend Message-ID: <20230703062925.GK3502534@coredump.intra.peff.net> In-Reply-To: On Tue, Jun 20, 2023 at 10:20:51AM -0400, Taylor Blau wrote: > Here is another reroll of my series to implement jump (née skip) lists > for the packed refs backend, based on top of the current 'master'. > > Most changes are minor, limited to changing variable names, moving > changes around between patches and tweaking commit messages for clarity. > I think that the first 9 or so patches are stable, but we may want some > more eyes on the remainder. I had been avoiding reading this series too carefully, as I was involved in many of the early patches. But now it's been long enough that I mostly forgot everything, so I could look at it with fresh eyes. ;) Plus the hidden-refs bits at the end were totally new to me. It looks good to me overall. I left a few small comments, some of which I think probably justify a re-roll. -Peff