<div dir="ltr">Same. 2-spaces hurts my little brain.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 10, 2018 at 12:50 PM Regina Obe <<a href="mailto:lr@pcorp.us">lr@pcorp.us</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div class="m_-4472508440992980245WordSection1"><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">Okay it appears I'm losing this game.<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">If we end up settling for spaces.  I'd like to request 4 spaces.  Cause with 2 spaces I'm too blind to see the space so it doesn't look indented at all..<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></p><div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt"><div><div style="border:none;border-top:solid #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in"><p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> geos-devel [mailto:<a href="mailto:geos-devel-bounces@lists.osgeo.org" target="_blank">geos-devel-bounces@lists.osgeo.org</a>] <b>On Behalf Of </b>Kurt Schwehr<br><b>Sent:</b> Monday, September 10, 2018 3:39 PM<br><b>To:</b> GEOS Development List <<a href="mailto:geos-devel@lists.osgeo.org" target="_blank">geos-devel@lists.osgeo.org</a>><br><b>Subject:</b> Re: [geos-devel] TABS or SPACES - ALL DEVELOPERS and PSC Vote<u></u><u></u></span></p></div></div><p class="MsoNormal"><u></u> <u></u></p><div><p class="MsoNormal">+1 to 2 or 4 spaces via a standard clang-format style<u></u><u></u></p><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal">I use multiple different editors/ides/code formatters and they all want to treat tabs differently :|<u></u><u></u></p></div></div><p class="MsoNormal"><u></u> <u></u></p><div><div><p class="MsoNormal">On Mon, Sep 10, 2018 at 11:57 AM Heiskanen Mika (FMI) <<a href="mailto:Mika.Heiskanen@fmi.fi" target="_blank">Mika.Heiskanen@fmi.fi</a>> wrote:<u></u><u></u></p></div><blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in"><div><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black">Hello,<u></u><u></u></span></p><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black"><u></u> <u></u></span></p><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black">Perhaps you should consider using automatic formatting tools such as clang-format?<u></u><u></u></span></p><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black">Just choose the style which is as close to your current style as possible, and then<u></u><u></u></span></p><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black">make as few changes to your .clang-format file as possible. Automatic tools will<u></u><u></u></span></p><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black">make your life much easier. Sometimes you will want to disable automatic formatting,<u></u><u></u></span></p><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black">and you can do that directly in the source code. You can make your editor automatically<u></u><u></u></span></p><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black">format the code, and you can add a git-hook to automatically format commits.<u></u><u></u></span></p><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black">​<u></u><u></u></span></p><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black">Mika<u></u><u></u></span></p><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black"><u></u> <u></u></span></p><p style="background:white"><span style="font-family:"Calibri",sans-serif;color:black"><u></u> <u></u></span></p><div><div class="MsoNormal" align="center" style="text-align:center;background:white"><span style="font-family:"Calibri",sans-serif;color:#212121"><hr size="2" width="98%" align="center"></span></div><div id="m_-4472508440992980245m_-8665434174537697754divRplyFwdMsg"><p class="MsoNormal" style="background:white"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:black">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:black"> geos-devel <<a href="mailto:geos-devel-bounces@lists.osgeo.org" target="_blank">geos-devel-bounces@lists.osgeo.org</a>> on behalf of Martin Davis <<a href="mailto:mtnclimb@gmail.com" target="_blank">mtnclimb@gmail.com</a>><br><b>Sent:</b> Monday, September 10, 2018 21:24<br><b>To:</b> GEOS Development List<br><b>Subject:</b> Re: [geos-devel] TABS or SPACES - ALL DEVELOPERS and PSC Vote</span><span style="font-family:"Calibri",sans-serif;color:#212121"> <u></u><u></u></span></p><div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121"> <u></u><u></u></span></p></div></div><div><div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121">My vote is the same as Vicky's - spaces, with indenting using multiples of 2. <u></u><u></u></span></p><div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121"><u></u> <u></u></span></p></div><div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121">Reasons:<u></u><u></u></span></p></div><div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121">- lots of religious discussion on the interweb about tabs vs spaces, but a quick read seems to indicate that the spaces camp is more popular<u></u><u></u></span></p></div><div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121">- spaces are more deterministic than tabs<u></u><u></u></span></p></div><div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121">- 2 rather than 4 gives more space for code<u></u><u></u></span></p></div><div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121"><u></u> <u></u></span></p></div><div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121">Martin<u></u><u></u></span></p></div><div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121"><u></u> <u></u></span></p></div></div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121"><u></u> <u></u></span></p><div><div><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121">On Mon, Sep 10, 2018 at 9:45 AM Regina Obe <<a href="mailto:lr@pcorp.us" target="_blank">lr@pcorp.us</a>> wrote:<u></u><u></u></span></p></div><blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in"><p class="MsoNormal" style="background:white"><span style="font-family:"Calibri",sans-serif;color:#212121">As many know the GEOS code base is a mess of different formatting.<br>Particular with tabs and spaces.<br>Our .editorconfig is equally messy in the name of compromise.<br><br>I'd like to make a Motion starting with GEOS 3.8 to reformat the .cpp, .h,<br>and .c<br><br>I think the other extensions are pretty standard and should stay as they<br>are.<br><br>That said, what would people prefer?<br><br>I vote:<br><br>TABS<br><br><br>Please all PSC and all developers who commit to GEOS voice an opinion.<br>We'll count the votes and go with the consensus with each vote counting<br>equally.<br>If any non-dev, non-PSC have an opinion, feel free to voice an opinion<br>keeping note we may not care about your opinion, but might possibly sway<br>ours :)<br><br>Once done - pramsey or cvvergara or strk will do a 1000 pound commit to<br>update all these files (including the editorconfig)<br><br>Thanks,<br>Regina<br><br>_______________________________________________<br>geos-devel mailing list<br><a href="mailto:geos-devel@lists.osgeo.org" target="_blank">geos-devel@lists.osgeo.org</a><br><a href="https://lists.osgeo.org/mailman/listinfo/geos-devel" target="_blank">https://lists.osgeo.org/mailman/listinfo/geos-devel</a><u></u><u></u></span></p></blockquote></div></div></div></div><p class="MsoNormal">_______________________________________________<br>geos-devel mailing list<br><a href="mailto:geos-devel@lists.osgeo.org" target="_blank">geos-devel@lists.osgeo.org</a><br><a href="https://lists.osgeo.org/mailman/listinfo/geos-devel" target="_blank">https://lists.osgeo.org/mailman/listinfo/geos-devel</a><u></u><u></u></p></blockquote></div><p class="MsoNormal"><br clear="all"><u></u><u></u></p><div><p class="MsoNormal"><u></u> <u></u></p></div><p class="MsoNormal">-- <u></u><u></u></p><div><p class="MsoNormal">--<u></u><u></u></p><div><p class="MsoNormal"><a href="http://schwehr.org" target="_blank">http://schwehr.org</a><u></u><u></u></p></div></div></div></div></div>_______________________________________________<br>
geos-devel mailing list<br>
<a href="mailto:geos-devel@lists.osgeo.org" target="_blank">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/mailman/listinfo/geos-devel</a></blockquote></div>