Re: [PATCHES] Escape handling in strings

On Jun 17, 2005, at 12:33 PM, Tom Lane wrote:

> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
>
>> OK, the current patch warns about two things, \' with one message,
>> and
>> any backslash in a non-E string with a different message.
>>
>
> Those are two very different things. \' is easy to get around and
> there's no very good reason not to send '' instead. But avoiding all
> use of \anything is impossible (think \\) so a non-suppressable
> warning
> for that would be quite unacceptable IMHO. I think it's much too
> early
> to be throwing a warning for \anything anyway. 8.2 or so, OK, but not
> in this cycle.

I think giving users a longer period of time to make the necessary
changes to their apps is very useful. If (as I understand) we're
giving them the opportunity to use E'' strings if they want to
continue to use \ for escaping, they can get rid of the warnings now,
by using E'' strings or using '' to escape. Getting people to
migrate something such as this is difficult and will take them quite
a while, I imagine. Giving them a longer time to change their
behavior as well as reinforcing it with a warning is helpful. They
can also easily check if they've got places they've missed in
changing their code, because the warnings will be prominent in their
logs.

Michael Glaesemann
grzm myrealbox com