Re: [PATCH 5/8] builtin/repack.c: extract `has_existing_non_kept_packs()`

From: Jeff King <peff@peff.net>
Date: Sep 7, 2023, 08:04 UTC
Message-ID: <20230907080428.GE1260718@coredump.intra.peff.net>
In-Reply-To: <e2cf87bb94da1a777f6b87ae3f32e036832839a6.1693946195.git.me@ttaylorr.com>

On Tue, Sep 05, 2023 at 04:36:51PM -0400, Taylor Blau wrote:

Show 7 quoted lines

> But a following change will store cruft- and non-cruft packs separately,
> meaning this check would break as a result. Prepare for this by
> extracting this part of the check into a new helper function called
> `has_existing_non_kept_packs()`.
> 
> This patch does not introduce any functional changes, but prepares us to
> make a more isolated change in a subsequent patch.

OK, that makes sense.

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