Re: [PATCHES] Escape handling in strings

From: "Andrew Dunstan" <andrew(at)dunslane(dot)net>
To: <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: <tgl(at)sss(dot)pgh(dot)pa(dot)us>, <andrew(at)dunslane(dot)net>, <pg(at)rbt(dot)ca>, <pgsql-hackers(at)postgresql(dot)org>, <chriskl(at)familyhealth(dot)com(dot)au>
Subject: Re: [PATCHES] Escape handling in strings
Date: 2005-06-17 01:03:04
Message-ID: 1293.24.211.165.134.1118970184.squirrel@www.dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Bruce Momjian said:

> OK, the current patch warns about two things, \' with one message, and
> any backslash in a non-E string with a different message. The \'
> message can easily be avoided in clients even in 8.0 by using '', but
> for E'', there is no way to prepare an application before upgrading to
> 8.1 because 8.0 doesn't have E''. (We can add E'' in a subrelease, but
> what percentage of users are going to upgrade to that?) This is why I
> think we need to add a GUC to allow the warning to be turned off. To
> be clear, the GUC is to control the warning, not the query behavior.
>
> We could go with the second warning only in 8.2, but that seems too
> confusing --- we should deal with the escape issue in two stages,
> rather than three.
>

So you don't agree with Tom's suggestion to implement E'' a full cycle
before removing backslash processing in standard strings? Or have I
misunderstood again?

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tim Allen 2005-06-17 01:04:33 Re: Autovacuum in the backend
Previous Message Josh Berkus 2005-06-17 00:31:37 DTrace Probes?

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2005-06-17 01:41:05 Re: [PATCHES] Escape handling in strings
Previous Message Bruce Momjian 2005-06-16 17:53:59 Re: fsync under win32