I just noticed when working on DDL deparsing that the typmodout routine
for intervals is broken.  The code uses

        if (precision != INTERVAL_FULL_PRECISION)
                snprintf(res, 64, "%s(%d)", fieldstr, precision);
        else
                snprintf(res, 64, "%s", fieldstr);


which puts the parenthised number after the textual name; but the
grammar only takes it the other way around.

This has been wrong since commit 5725b9d9afc8 dated Dec 30 2006, which
introduced the whole notion of type-specific typmod output functions.
I don't understand how come nobody has noticed this in eight years.

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to