Re: [PATCH 2/4] repack: populate extension bits incrementally

From: Taylor Blau <me@ttaylorr.com>
Date: Oct 21, 2022, 23:41 UTC
Message-ID: <Y1MuMJT0ntvoTeUZ@nand.local>
In-Reply-To: <Y1MsdAgL8fdIRtxH@coredump.intra.peff.net>

On Fri, Oct 21, 2022 at 07:34:12PM -0400, Jeff King wrote:

Show 8 quoted lines

> > which still lets you eagerly keep track of the generated pack extensions
> > while also protecting against forgetful callers. Obviously we're relying
> > on a runtime check which is going to be somewhat weaker. But I think
>
> I don't think we need that. The renaming loop a few lines below will
> happily segfault if anybody forgot to populate it. With a less nice
> message, obviously, but if the point is to notice a bug, it will get the
> job done.

That's reasonable, I think. I'm happy to abandon that suggestion :-).

Thanks, Taylor

$ 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.