Quoting Garrett Rooney <[EMAIL PROTECTED]>:
I guess I'm not seeing how adding a bunch of new format strings isn't changing anything... Am I just misunderstanding how the patch works? It looks like it replaces the existing format string parsing with all new parsing. Is it just that the new parsing is a superset of the existing stuff?
Yes, that's how it should work - as a superset. There are new format strings that we would support, apart from already supported ones, which were %s and %d (only in PostgreSQL ATM). The %s and %d would remain with the same meaning and anything that isn't specifically defined would still be interpreted as %s (i.e. VARCHAR), just like it is now (although, I find that to be a bad idea - we should stick to explicit meanings).
The parsing would get moved out of the drivers, so that we can rely on it being identical for all drivers. Which would make things a bit more consistent.
-- Bojan
