Robert Haas writes:
> On Mon, Jun 4, 2012 at 8:48 PM, Tom Lane wrote:
>> And so is that. IMO the error reporting in this module could stand to
>> be reviewed altogether for compliance with our message guidelines.
>> (For starters, why is it using errdetail_internal?) I refrained from
>> editori
On Mon, Jun 4, 2012 at 8:48 PM, Tom Lane wrote:
>> In addition, error message above seems corrupted in my environment.
>> Here we check not-escaped control character, so printing it with %c
>> formatting might break log files. How about using decimal or hex dump
>> in such cases?
>
> And so is th
Shigeru Hanada writes:
> json_lex_string() misjudges that the token "ãã¼" contains naked
> (not-escaped) control character, because its first byte is 0xe3 and it's
> -29 for signed char interpreting, and it's less than 32. We need to
> cast to unsigned char (or use unsigned variable) here.
Ye
