<div dir="auto"><div>+1 for stop f**king with formatting.</div><div dir="auto"><br></div><div dir="auto">I wonder when we'll get to debating camel vs snake.<br><br><div data-smartmail="gmail_signature" dir="auto">-bborie</div><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">On Feb 28, 2018 08:15, "Paul Ramsey" <<a href="mailto:pramsey@cleverelephant.ca">pramsey@cleverelephant.ca</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1. The issue is additional dependencies and minimal effort for devs<br>
to create identical results. An embedded formatter like astyle or<br>
pgindent gives us the easiest way of achieving that. (Another proposed<br>
solution: stop f___ing with formatting. I know, it's an impossible<br>
dream.)<br>
<font color="#888888"><br>
P.<br>
</font><div class="elided-text"><br>
On Wed, Feb 28, 2018 at 8:09 AM, Sandro Santilli <<a href="mailto:strk@kbt.io">strk@kbt.io</a>> wrote:<br>
> On Wed, Feb 28, 2018 at 02:34:08PM +0000, Darafei "Komяpa" Praliaskouski wrote:<br>
><br>
>> TLDR: pgindent heavily broken, style ported to clang-format, let's use<br>
>> clang-format with pgindent style.<br>
><br>
> Maybe you failed to understand the reason "pgindent" was considered<br>
> better than "clang-format": possibility to shipt the tool with PostGIS.<br>
><br>
> If "pgindent" is broken I suggest to file issues upstream:<br>
><br>
>     <a href="https://git.postgresql.org/git/pg_bsd_indent.git" rel="noreferrer" target="_blank">https://git.postgresql.org/<wbr>git/pg_bsd_indent.git</a><br>
><br>
> According to Tomas this would be the same code as the one also<br>
> embedded in PostgreSQL, but maybe that's not the case ?<br>
><br>
>     <a href="https://github.com/postgres/postgres/tree/master/src/tools/pgindent" rel="noreferrer" target="_blank">https://github.com/postgres/<wbr>postgres/tree/master/src/<wbr>tools/pgindent</a><br>
><br>
>> Some practices from pgindent and postgres code stylestill can be adopted<br>
>> though even the tool does not fit. I ported Postgres brace style to<br>
>> .clang-format in my PR.<br>
><br>
> I really don't care about which style is choosen, as long as we can<br>
> make it easy for any contributor to produce patches that will meet<br>
> the criteria our build bots will expect.<br>
><br>
> At the moment the proposed solutions are:<br>
><br>
>      tool              | additional dependencies<br>
>   ---------------------+--------<wbr>-----------------------------<br>
>    embedded pgindent   |<br>
>   ---------------------+--------<wbr>-----------------------------<br>
>    embedded astyle     | a C++ compiler<br>
>   ---------------------+--------<wbr>-----------------------------<br>
>    clang-format        | clang-format of a specific version<br>
>   ---------------------+--------<wbr>-----------------------------<br>
><br>
> --strk;<br>
><br>
>   ()   Free GIS & Flash consultant/developer<br>
>   /\   <a href="https://strk.kbt.io/services.html" rel="noreferrer" target="_blank">https://strk.kbt.io/services.<wbr>html</a><br>
> ______________________________<wbr>_________________<br>
> postgis-devel mailing list<br>
> <a href="mailto:postgis-devel@lists.osgeo.org">postgis-devel@lists.osgeo.org</a><br>
> <a href="https://lists.osgeo.org/mailman/listinfo/postgis-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/postgis-devel</a><br>
______________________________<wbr>_________________<br>
postgis-devel mailing list<br>
<a href="mailto:postgis-devel@lists.osgeo.org">postgis-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/postgis-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/postgis-devel</a></div></blockquote></div><br></div></div></div>