Re: [PATCH 12/12] builtin/*: remove or amend "USE_THE_INDEX_COMPATIBILITY_MACROS"

From: Taylor Blau <me@ttaylorr.com>
Date: Nov 18, 2022, 18:42 UTC
Message-ID: <Y3fSJwm7qoVmvPA8@nand.local>
In-Reply-To: <patch-12.12-b69cf6d7600-20221118T112205Z-avarab@gmail.com>

On Fri, Nov 18, 2022 at 12:31:06PM +0100, Ævar Arnfjörð Bjarmason wrote:

Show 6 quoted lines

> Follow-up the preceding commit and change
> "USE_THE_INDEX_COMPATIBILITY_MACROS" to the more narrow
> "USE_THE_INDEX_VARIABLE" in cases where we still use "the_index".
>
> Then remove "USE_THE_INDEX_VARIABLE" entirely in the few cases where
> we don't require any compatibility macros or variables anymore.

Could this get squashed into the previous commit? IOW, is there any reason to keep them separate?

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.