<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">My 2 cents:<br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I don't like automatic formatters they are "code invasive".<br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">What I use is cpplint, and its configurable, which does not format the code, it only indicates where a formating style is not followed.<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Its coded in python, some not all of the google code C++ style guide is followed.<br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">So, how I worked during the pgRouting rewrite (once I decided to use it) was:<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">use cpplint on the new or modified files.<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">No bug to fix, leave the file in peace.<br>No additional code to add, leave the file in peace.<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br>Reading the C code of postGIS, (even that are huge files) its very nice.<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Interesting would be if based on what is there, you configure cpplint to have to make the least amount of changes on your code.<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">[1] <a href="https://github.com/pgRouting/pgrouting/blob/master/tools/scripts/code_checker.sh">https://github.com/pgRouting/pgrouting/blob/master/tools/scripts/code_checker.sh</a><br>[2] <a href="https://google.github.io/styleguide/cppguide.html">https://google.github.io/styleguide/cppguide.html</a><br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br><br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br><br><br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 5, 2018 at 10:19 AM, Sandro Santilli <span dir="ltr"><<a href="mailto:strk@kbt.io" target="_blank">strk@kbt.io</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat, Mar 03, 2018 at 01:50:51PM -0500, Regina Obe wrote:<br>
<br>
> From my perspective, the 3 most important things about consistent convention are:<br>
><br>
> 1) Tab or Space for indenting (and if space how many spaces)<br>
> 2) LF or CRLF for line end<br>
> 3) character set (this has bitten us in the past with people editing the NEWS file and changing the encoding to Latin or something and screwing up people's names)<br>
<br>
</span>I agree, and I think editorconfig is doing all of the above.<br>
<br>
For the record, I also saw the editorconfig plugin for Vim<br>
strip trailing blanks from untouched lines. It is the main<br>
reason why you find some "strip trailing slashes" commits from<br>
myself in the relatively recenty codechange history.<br>
I do not remember it changing indentation from non-touched lines).<br>
<br>
--strk;<br>
<div class="HOEnZb"><div class="h5">______________________________<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></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><pre>Georepublic UG (haftungsbeschränkt)
Salzmannstraße 44, 
81739 München, Germany

Vicky Vergara
Operations Research

eMail: vicky@<a href="http://georepublic.de" target="_blank">georepublic.de</a>
Web: <a href="https://georepublic.info" target="_blank">https://georepublic.info</a>

Tel: +49 (089) 4161 7698-1
Fax: +49 (089) 4161 7698-9

Commercial register: Amtsgericht München, HRB 181428
CEO: Daniel Kastl

<span></span></pre></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>
</div>