Re: [PATCH 12/17] odb: trivial refactorings to get rid of `the_repository`

From: Derrick Stolee <stolee@gmail.com>
Date: May 7, 2025, 01:25 UTC
Message-ID: <728e13e3-b563-4d66-bc00-b3320b76ec9f@gmail.com>
In-Reply-To: <20250506-pks-object-store-wo-the-repository-v1-12-c05b82e7b126@pks.im>

On 5/6/25 7:09 AM, Patrick Steinhardt wrote:

Show 7 quoted lines

> All of the external functions provided by the object database subsystem
> don't depend on `the_repository` anymore, but some internal functions
> still do. Refactor those cases by plumbing through the repository that
> owns the object database.
> 
> This change allows us to get rid of the `USE_THE_REPOSITORY_VARIABLE`
> preprocessor define.

Show 5 quoted lines

> --- a/odb.c
> +++ b/odb.c
> @@ -1,5 +1,3 @@
> -#define USE_THE_REPOSITORY_VARIABLE
> -

Very satisfying! Thanks, -Stolee

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