<div dir="ltr">Mateusz just created <a href="https://trac.osgeo.org/gdal/wiki/rfc69_cplusplus_formatting">https://trac.osgeo.org/gdal/wiki/rfc69_cplusplus_formatting</a> and I'll be cloning his <a href="https://trac.osgeo.org/geos/wiki/RFC4">https://trac.osgeo.org/geos/wiki/RFC4</a> with changes for GDAL.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 4, 2017 at 1:49 PM, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@spatialys.com" target="_blank">even.rouault@spatialys.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><u></u>
<div style="font-family:'monospace';font-size:9pt;font-weight:400;font-style:normal">
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Hi,</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">I've observed Kurt's continued efforts, over several months if not years, to set a consistant style to the code base and this is a much appreciated initiative. Now, as this touches the whole code base, I think this should be formalized, probably under the form of a RFC, so that this is shared among all developers. Currently as there's no automation, basically every committer will probably unconsciously "break" files that have been already reformatted, which is a waste of time that could be avoided (and on a minor note we could save extra reformatting commits that are noise in the history, and extra bits to parse for the human preparing a changelog)</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">I've seen in other projects, like QGIS, a script (in QGIS, they rely on astyle ultimately. not saying we should necessary use astyle) that takes care of reformatting automatically the files to be committed given the rules. That way the effort is spread on all contributors (and it is not a big deal to run this script). We could also have in Travis-CI a check so that pull requests are verified to respect the style (currently, we have already a very adhoc scripts/detect_tabulations.sh to check that no tabulation characters are used)</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Even</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">-- </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Spatialys - Geospatial professional services</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><a href="http://www.spatialys.com" target="_blank">http://www.spatialys.com</a></p></div><br>______________________________<wbr>_________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/gdal-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">--<div><a href="http://schwehr.org" target="_blank">http://schwehr.org</a></div></div>
</div>