On 21/03/2008, Tom Lane <[EMAIL PROTECTED]> wrote:
> "Brendan Jurd" <[EMAIL PROTECTED]> writes:
>
> > As discussed on -hackers, this patch allows the construction of an
>  > empty array if an explicit cast to an array type is given (as in,
>  > ARRAY[]::int[]).
>
>
> Applied with minor fixes; mostly, ensuring that the cast action would
>  propagate down to sub-arrays, as in

Great, thanks Tom.

>  I was interested to realize that this fix validates the decision to
>  pass down the type information on-the-fly during transformExpr recursion.
>  It would have been a lot more painful to do it if we'd taken the A_Const
>  approach.
>

Indeed.

>  I didn't do anything about removing A_Const's typename field, but I'm
>  thinking that would be a good cleanup patch.
>

I'd be happy to take this on.  My day job is pretty busy at the moment
but I should be able to submit something in a week or so.

Cheers,
BJ

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

Reply via email to