git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] use C99 declaration of variable in for() loop

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 15, 2024, 20:07 UTC
Message-ID
<xmqqmss1pk4a.fsf@gitster.g>
In-Reply-To
<CA+EOSBktLGnzi+DjOTVv9_cVNsM_fjEKiF9kWnaYrGtvGJ-Kdg@mail.gmail.com>
Elia Pinto <gitter.spiros@gmail.com> writes:
> Thank you for your reply. I can understand that, clearly. However,
> this means that extensive code
> refactoring contributions are never welcome. I am not saying this is a
> problem, but just an observation.

Such changes can happen and have happened when the benefit of such code churn outweighs the cost of reviewing *and* cost of updating or adjusting in-flight topics that may already or may not yet be in my tree. Coccinelle-driven patches that can be mechanically reproduced and whose validity can be trusted can be one way to reduce the review and maintenance cost for such a tree-wide change.

Previous: Elia PintoNext: brian m. carlson
Message 6 of 8 in “use C99 declaration of variable in for() loop”
  1. use C99 declaration of variable in for() loopElia Pinto, Feb 15, 2024
  2. Junio C HamanoFeb 15, 2024
  3. Christian CouderFeb 15, 2024
  4. Junio C HamanoFeb 15, 2024
  5. Elia PintoFeb 15, 2024
  6. Junio C HamanoFeb 15, 2024
  7. brian m. carlsonFeb 15, 2024
  8. Junio C HamanoFeb 15, 2024

Read the whole thread, see it on lore, or plain text.

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