Re: [PATCH v2 0/3] remote: replace static variables with struct remote_state

From: Glen Choo <chooglen@google.com>
Date: Oct 13, 2021, 22:00 UTC
Message-ID: <kl6lr1co1t62.fsf@chooglen-macbookpro.roam.corp.google.com>
In-Reply-To: <xmqq8rywfz6x.fsf@gitster.g>

Junio C Hamano <gitster@pobox.com> writes:

> I am not you, so I can only speculate, but the real reason _could_
> be that it makes it simpler to formulate steps 2 and 3 mechanically.

This is somewhat true, however...

> I think that approach would make more work to get to the final
> state, though.

I found that if I were to add the backpointer, it would have been better to add it early and avoid this extra work that you mention here :)

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