Tom Lane wrote:
The quoting logic is still completely the wrong thing :-(.  For one
thing, quote_literal will try to generate E'' syntax in some cases.
But more to the point, quote_literal's quoting rules don't match
what is needed.  A look at libpq's conninfo_parse says that what it
accepts is single-quoted strings in which backslash quotes the next
character.  It does not recognize doubled single quotes.  I think
you will need to whip up a special-purpose quoting subroutine.

OK, I see that. I assume I need to care for encoding issues? If so, do I assume server encoding or client encoding?


+               return pstrdup(buf->data);

is a waste of time.  The StringInfo's buffer is already palloc'd.

Thanks -- will fix.

Joe

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to