On 5 May 2017 at 13:35, Even Rouault <even.roua...@spatialys.com> wrote: > > In case we want to have a policy regarding braces around single statements, > one finding coming from the openjpeg discussion I mentionned : > > """ > Moreover, concerning braces around single statements, it seems astyle does > this with the add-braces option: > http://astyle.sourceforge.net/astyle.html#_add-braces > As far as I understood, clang-format does not, and requires another tool > (clang-tidy) to deal with these cases. So one more point for astyle. > """
Rather, it is a separation of concerns: linting vs formatting AFAIU, clang-format does not modify any code (ie. add tokens). I'm not clang-format advocate. I'll repeat myself: My personal preference is to use clang-format. I proposed it to GEOS via GEOS RFC. Kurt found the RFC useful and is using it as a template to prepare one for GDAL. IMHO, at the end of the day, tool does not matter as long as proposed goals are achieved. Best regards, -- Mateusz Loskot, http://mateusz.loskot.net _______________________________________________ gdal-dev mailing list gdal-dev@lists.osgeo.org https://lists.osgeo.org/mailman/listinfo/gdal-dev
