From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Joe Conway <mail(at)joeconway(dot)com> |
Cc: | "Hackers (PostgreSQL)" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [Fwd: Re: dblink patches for comment] |
Date: | 2009-06-06 16:08:23 |
Message-ID: | 10658.1244304503@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Joe Conway <mail(at)joeconway(dot)com> writes:
> Based on Tom's post today about RC1, it sounds like I need to get this
> committed very soon. Any complaints?
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.
One other really minor point is that the pstrdup here:
> + return pstrdup(buf->data);
is a waste of time. The StringInfo's buffer is already palloc'd.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2009-06-06 16:42:01 | Re: PostgreSQL Developer meeting minutes up |
Previous Message | Markus Wanner | 2009-06-06 15:41:10 | Re: PostgreSQL Developer meeting minutes up |