Roman Khapov <[email protected]> writes:
> I noticed, that there is bug in escaping values that contains '\' or '"' in
> text representation
> inside pageinspect for gist: the string 'foo"bar' are printed like "foo""bar"
> and not "foo\"bar".
I do not think this is a bug. The comment at line 295 says
"Most of this is copied from record_out().", and this logic
matches what record_out() does, and the output is legal
according to the manual's specifications [1]:
To put a double quote or backslash in a quoted composite field
value, precede it with a backslash. (Also, a pair of double quotes
within a double-quoted field value is taken to represent a double
quote character, analogously to the rules for single quotes in SQL
literal strings.)
Now, your alternative coding would also produce legal output, but
I do not think unnecessary change here is a good thing.
regards, tom lane
[1] https://www.postgresql.org/docs/devel/rowtypes.html#ROWTYPES-IO-SYNTAX
