Re: [PATCH v2 00/17] Header cleanups

From: Derrick Stolee <derrickstolee@github.com>
Date: Feb 27, 2023, 13:51 UTC
Message-ID: <d130d7df-313f-bb6f-524c-d1204c0ce627@github.com>
In-Reply-To: <pull.1485.v2.git.1677197376.gitgitgadget@gmail.com>

On 2/23/2023 7:09 PM, Elijah Newren via GitGitGadget wrote:

Show 9 quoted lines

> Changes since v1:
> 
>  * Fixed a bad squash; moved the changes to the right commit
>  * More thorough justification in the commit message for the 2nd commit
>  * Removed an accidental added include of gettext.h (the file was used, but
>    it makes it harder to review that patch)
>  * Add some explanation of surprising changes to commit messages
>  * Add a new commit which removes the include of hex.h from cache.h, and
>    instead makes C files include it directly, as suggested by Stolee

I'm happy with these updates. Thanks for the series!

-Stolee

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