Re: [PATCH 1/8] packfile: use a `strmap` to store packs by name

From: Taylor Blau <me@ttaylorr.com>
Date: Oct 29, 2025, 22:16 UTC
Message-ID: <aQKSMk6nFIk6Xomh@nand.local>
In-Reply-To: <20251028-pks-packfiles-store-drop-list-v1-1-1a3b82030a7a@pks.im>

On Tue, Oct 28, 2025 at 12:08:31PM +0100, Patrick Steinhardt wrote:

> ---
>  packfile.c | 24 ++++--------------------
>  packfile.h |  4 ++--
>  2 files changed, 6 insertions(+), 22 deletions(-)

Nice; well explained and the change below looks obviously correct to me. I much prefer the strmap API here and agree that this is a good use-case for it.

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.