<div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>> > <a href="https://github.com/postgis/postgis/blob/svn-trunk/STYLE#L27" rel="noreferrer" target="_blank">https://github.com/postgis/postgis/blob/svn-trunk/STYLE#L27</a><br>
>><br>
>>  I'm perfectly fine with spirit, though others may feel differently.<br>
<br>
> I'm fine with the spirit, it's just the implementation which lacks:<br>
> when we tried, the "run astyle" step produced different output for different people.<br>
How about we don't run astyle or just run once in a while (as we've been doing) and have someone tasked to do it stuck with some version of astryl, or just relegate it to a bot.<br></blockquote><div><br></div><div>I volunteer to commit to maintain consistent PostGIS codebase formatting as responsibility in project.</div><div><br></div><div>That would save everyone the hassle of thinking about styling, finding exact version numbers of software they don't care about, and get no frustration of broken CI because of code white space and not logic.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'd rather not even have to have the hassle of running astyle.  I was all happy with editorconfig as it did not<br>
slow down my workflow or require one more thing for my fading memory to remember to do.<br>
<br>
I'm still lost at this obsession of perfectly consistent formatting. </blockquote><div><br></div><div>Perfectly consistent formatting just comes as a freebie when you use a formatter.</div><div><br></div><div>The main reason I formatted lwgeom_geos.c file completely was it having unmatched indentation levels that are hard to find manually.</div><div>After you meet first one of them you're not really sure whether the rest of the performs in the way it is formatted, or a { hides somewhere between #ifdefs and changes bounds of a block.<br><br></div><div>Such things are probably not an issue for original authors as they remember what they were doing, and it works in rough "one developer per subsystem" model, but breaks as you get more people fixing old issues on board.<br></div></div></div>