2013/1/28 Tom Lane <[email protected]>: > Pavel Stehule <[email protected]> writes: >> 2013/1/28 Dean Rasheed <[email protected]>: >>> Starting with the first patch - it issues a new WARNING if the format >>> string contains a mixture of format specifiers with and without >>> parameter indexes (e.g., 'Hello %s, %1$s'). >>> >>> Having thought about it a bit, I really don't like this for a number of >>> reasons: > >> I am not sure what you dislike? >> warnings or redesign of behave. > > Both. If we had done this when we first implemented format(), it'd be > fine, but it's too late to change it now. There very likely are > applications out there that depend on the current behavior. As Dean > says, it's not incompatible with SUS, just a superset, so ISTM this > patch is proposing to remove documented functionality --- for no very > strong reason.
I disagree - but I have not a arguments. I am thinking so current implementation is wrong, and now is last time when we can to fix it - format() function is not too old and there is relative chance to minimal impact to users. I didn't propose removing this functionality, but fixing via more logical independent counter for ordered arguments. Dependency on previous positional argument is unpractical and unclean. I am not satisfied so it is undefined and then it is ok. Regards Pavel > > I vote for rejecting this change entirely. > > regards, tom lane -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
