Re: [PATCH v7 15/20] submodule--helper: move functions around

From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Feb 12, 2022, 14:41 UTC
Message-ID: <220212.86leygxi3t.gmgdl@evledraar.gmail.com>
In-Reply-To: <20220210092833.55360-16-chooglen@google.com>

On Thu, Feb 10 2022, Glen Choo wrote:

> A subsequent commit will change the internals of several functions and
> arrange them in a more logical manner. Move these functions to their
> final positions so that the diff is smaller.

Shouldn't we do this earlier & avoid the FIXME comment in 10/20? (maybe a subsequent fixup addresses it, haven't checked...)

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