From: Johannes Schindelin Date: Fri, 10 Oct 2025 13:55:55 GMT Subject: Re: [PATCH 0/2] Organize mingw includes Message-ID: In-Reply-To: Hi Patrick, On Fri, 10 Oct 2025, Patrick Steinhardt wrote: > On Thu, Oct 09, 2025 at 07:45:59AM +0000, Johannes Schindelin via GitGitGadget wrote: > > Following in the footsteps of the many, many recent #include refactorings, > > this patch series orders the #include statements in compat/mingw.c. > > Both of these patches look good to me and I like the improved > consistency that they bring. I may also do the same for our code in > "refs/", where some of the files use relative includes, as well. Might > be worth to document this somewhere if it isn't already, but that > doesn't have to be part of your patch series here. There's already such a lot of documentation that I fear it is intimidating to any new contributor (and if I was one, I'd be tempted to read it exclusively via AI summaries). I don't want to add to that amount. > Sorting them also makes sense. It's another thing where I wish that we > had a tool to enforce this. clang-format supports this in theory, but > it's disabled right now. And I'm not even sure whether it can be told to > include e.g. "git-compat-util.h" first. In theory, I am totally with you: Sorting `#include`s is a job best left to tools. But then, I say the same about formatting, and I see a lot of appetite on this list for preventing that to become a tool-driven job, and instead requiring developer time to be spent on it. Therefore, I don't want to spend any effort trying to get this automated when I see small chances of success in the endeavor of freeing up cognitive cycles for tasks I find more intellectually rewarding than figuring out where a space or a line break goes. Ciao, Johannes