Re: [PATCH 0/5] refs: remove functions without ref store

From: Junio C Hamano <gitster@pobox.com>
Date: May 3, 2024, 18:24 UTC
Message-ID: <xmqq7cga7nzo.fsf@gitster.g>
In-Reply-To: <20240503173553.GC3631237@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

> Though maybe an even more radical proposal: now that read_ref_full(),
> etc, are gone, and we have only refs_read_ref_full(), could/should we
> shorten the latter to drop the "refs_" prefix?

I view it as a good longer-term goal. But I also view it as an orthogonal issue to the transition.

We need a smooth migration path for remaining callers of these older functions. We could do the USE_THE_INDEX_MACROS like compatibility layer during transition period.

Thanks.

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