[postgis-devel] Code formatting digging in
Sandro Santilli
strk at kbt.io
Thu Feb 22 08:29:52 PST 2018
Hi Darafei,
On Thu, Feb 22, 2018 at 07:52:47AM +0000, Darafei wrote:
> New Revision: 16409
>
> Modified:
> trunk/liblwgeom/lwgeom_geos_clean.c
> Log:
> Format lwgeom_geos_clean.c before digging in
>
> References #2508
[...]
> - switch ( GEOSGeomTypeId(g_in) )
> + switch (GEOSGeomTypeId(g_in))
Here you add spaces
> - if ( ret ) return ret;
> + if (ret) return ret;
Here you remove spaces
What's the rationale for this style change ?
I think we should try not to change style unless we have
a way to enforce a specific style always, and this is so
far only done by the settings in .editorconfig.yml, which
do not address spacing in conditions and switches...
--strk;
More information about the postgis-devel
mailing list