Re: [PATCH v2 0/8] repack: support repacking into a geometric sequence

From: Jeff King <peff@peff.net>
Date: Feb 17, 2021, 00:01 UTC
Message-ID: <YCxcyQcNO9LO2n9m@coredump.intra.peff.net>
In-Reply-To: <cover.1612411123.git.me@ttaylorr.com>

On Wed, Feb 03, 2021 at 10:58:45PM -0500, Taylor Blau wrote:

> The details of the new approach can be found in the third patch, but the gist is
> as follows:
> [...]

I think this turned out very nice (and less complicated than I feared it might). I've read up through patch 5. I think the overall approach is good, but I had various small-to-medium comments.

I'll try to pick up reviewing the rest tomorrow, though it may make sense to resolve the earlier comments first.

-Peff

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.