<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I am using  code_linter locally<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">python code_linter/styleguide/cpplint/cpplint.py --filter=-whitespace/tab src/operation/polygonize/PolygonizeGraph.cpp<br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 23, 2018 at 1:09 PM, Paul Ramsey <span dir="ltr"><<a href="mailto:pramsey@cleverelephant.ca" target="_blank">pramsey@cleverelephant.ca</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 Sun, Jul 22, 2018 at 2:13 PM, Vicky Vergara <<a href="mailto:vicky@georepublic.de">vicky@georepublic.de</a>> wrote:<br>
> I wanted to start with changing the for loops to range loops, and chose<br>
> PolygonizeGraph class as first C++ class where to do the changes.<br>
> but when I started doing that small change, I just couldn't stop myself from<br>
> using c++11 goodies on PolygonizeGraph.<br>
> Now the file does not look at all as the original (big smile)<br>
> java is not C++, so C++ does not need to look like java, now, in my opinion,<br>
> the code on PolygonizeGraph looks more like C++.<br>
><br>
> <a href="https://github.com/libgeos/geos/compare/master...cvvergara:PolygonizeGraph" rel="noreferrer" target="_blank">https://github.com/libgeos/<wbr>geos/compare/master...<wbr>cvvergara:PolygonizeGraph</a><br>
<br>
</span>As Sandro noted, the more we drift from the Java the more "fun"<br>
porting changes will be.<br>
WRT your branch, the mixture of formatting with actual changes makes<br>
it hard to find the real work.<br>
Also, you're imposing your syntactical style rather than matching it,<br>
things like indentation, etc, etc.<br>
I understand the mental place you're coming from doing that, I come<br>
from that place too, but since everyone has different styles, can you<br>
stick to real changes, and we can all agree (?) on a post-commit<br>
formatter?<br>
<span class="HOEnZb"><font color="#888888">P.<br>
</font></span><div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
geos-devel mailing list<br>
<a href="mailto:geos-devel@lists.osgeo.org">geos-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/geos-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/geos-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>