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

Bruce Momjian <pgman@candle.pha.pa.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



---------------------------(end of broadcast)---------------------------
TIP 4: Don't 'kill -9' the postmaster

Reply via email to