[postgis-devel] Code formatting digging in

Paul Ramsey pramsey at cleverelephant.ca
Thu Feb 22 08:33:14 PST 2018


Isn't he removing spaces in both?
It looks like he doesn't like my old quirk of padding the conditional
bits. In fairness, it seems to be a minority position in the C
formatting community.
Shall we do another 12 rounds on auto-formatters?
P

On Thu, Feb 22, 2018 at 8:29 AM, Sandro Santilli <strk at kbt.io> wrote:
> 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;
> _______________________________________________
> postgis-devel mailing list
> postgis-devel at lists.osgeo.org
> https://lists.osgeo.org/mailman/listinfo/postgis-devel


More information about the postgis-devel mailing list