On 01/02/2013 04:45 PM, Robert Haas wrote:
On Wed, Dec 26, 2012 at 3:33 PM, Andrew Dunstan <and...@dunslane.net> wrote:
Here is a patch for the first part of the JSON API that was recently
discussed. It includes the json parser hook infrastructure and functions
for json_get and friends, plus json_keys.
As is, this exposes the json lexer fully for use by the hook functions. But
I could easily be persuaded that this should be an opaque structure with
some constructor and getter functions - I don't think the hook functions
need or should be able to set anything in the lexer.
Work is proceeding on some of the more advanced functionality discussed.
This seems to contain a large number of spurious whitespace changes.
I'm glad you're looking at it :-)
I did do a run of pgindent on the changed files before I cut the patch,
which might have made some of those changes.
I notice a couple of other infelicities too, which are undoubtedly my fault.
The original prototype of this was cut against some older code, and I
then tried to merge it with the current code base, and make sure that
all the regression tests passed. That might well have resulted in a
number of things that need review.
And maybe some other spurious changes. For example, I'm not sure why
this comment is truncated:
Another good question.
I'll make another pass over this and try to remove some of what's
annoying you.
cheers
andrew
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
