Re: pdo: patch to make leading colon unneeded for bound variables without breaking BC.

Wez Furlong wrote:

There are releases on pecl.php.net, and there are (a few) people running these in production. In addition, it's been advertised as working this way for over a year. So, it is important to preserve the current behaviour when this patch is applied (will do so this weekend; need to catch up on a lot of things first).

Hehe, this is too funny. Last time you dismissed my BC concerns when I said we should remove ":" now by saying something along the lines, we can always make things less strict. If this does not make sense now its because it never did :)

I've had code running using ':' since before this topic was first raised. I don't see what's funny about it; the fact is that there is code alive using : prefixes; we can't simply stop supporting it. So the way things have been 'standardised' currently are cast in stone, even if maintaining them introduces complex checks across other drivers?

Thies' patch is simply "making things less strict", and that is not something I had ruled out, just a decision I wanted to defer. I would have thought the reference should be the SQL standard rather than "making things less strict" - Having to ADD the ':' is something I don't remember having to do for some time - in production.

Since available time is short, playing with pdo is still being deferred - at least until I can get pdo_firebird to do anything useful ;) -- Lester Caine ----------------------------- L.S.Caine Electronic Services