Re: [PATCH 15/16] packfile: refactor `get_all_packs()` to work on packfile store

From: Patrick Steinhardt <ps@pks.im>
Date: Aug 21, 2025, 06:40 UTC
Message-ID: <aKa_erHX3aFhgL5j@pks.im>
In-Reply-To: <CAOLa=ZTZzomWcCObMoBtNzFzNrC-OCWuzTqkTUB5+xnJ5VoV1g@mail.gmail.com>

On Wed, Aug 20, 2025 at 06:53:34AM -0700, Karthik Nayak wrote:

Show 10 quoted lines

> Patrick Steinhardt <ps@pks.im> writes:
> 
> > The `get_all_packs()` function prepares the packfile store and then
> > returns its packfiles. Refactor it to accept a packfile store instead of
> > a repository to clarify its scope.
> >
> 
> [snip]
> 
> Nit: From running the clang formatter, small cleanups:

These would all be while-at-it changes though, so I'd prefer to punt on them for now.

Patrick

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