Tom Lane escribió:
> alvherre@[EMAIL PROTECTED]
(Alvaro Herrera) writes:
> > Remove typename from A_Const.
>
> I'm thinking this could be cleaned up further. The patch as applied
> removes the ::int4 typename decoration that had been inserted by
> makeIntConst(), while leaving in place the ::float8 decoration inserted
> by makeFloatConst(). The kindest thing that can be said about that
> is that it's inconsistent.
That's very kind, yes :-) I think that cast can be removed safely.
> Now as far as I can see in a look through
> gram.y, these routines (and makeAConst) were used only in places where
> the typename qualification was really unnecessary, that is typmods
> and GUC variable values and so on, not general expression contexts where
> we might really need to determine a data type for the constant. So what
> I'm thinking is that we should get rid of the ::float8 decoration too,
> and thereby be able to revert some of the ugly code added elsewhere such
> as guc.c.
Hmm, I'm not sure but I think the typecast is needed in the guc.c code
in order to pass the fact that the Const is an Interval. This is used
to process things such as
set time zone interval '-8:30' ;
Perhaps this can be detected by some other mechanism but currently it's
being driven by the cast.
--
Alvaro Herrera
http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 sup****t
--
Sent via pgsql-hackers mailing list (pgsql-hackers@[EMAIL PROTECTED]
)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
|
