Re: new json funcs - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: new json funcs
Date
Msg-id 52DAE652.2050209@dunslane.net
Whole thread Raw
In response to Re: new json funcs  (Marko Tiikkaja <marko@joh.to>)
Responses Re: new json funcs
List pgsql-hackers
On 01/18/2014 12:34 PM, Marko Tiikkaja wrote:
> On 2014-01-17 03:08, Andrew Dunstan wrote:
>> In case anyone feels like really nitpicking, this version fixes some
>> pgindent weirdness due to an outdated typedef list in the previous 
>> version.
>
> Is it possible for you to generate a diff that doesn't have all these 
> unrelated changes in it (from a pgindent run, I presume)?  I just read 
> through three pagefuls and I didn't see any relevant changes, but I'm 
> not sure I want to keep doing that for the rest of the patch.
>
>
>


This seems to be quite overstated. The chunks in the version 3 patch 
that only contain pgindent effects are those at lines 751,771,866 and 
1808 of json.c, by my reckoning. All the other changes are more than 
formatting.

And undoing the pgindent changes and then individually applying all but 
those mentioned above would take me a lot of time.

cheers

andrew



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [PATCH] Make various variables read-only (const)
Next
From: Marko Tiikkaja
Date:
Subject: Re: new json funcs