On 21/03/2008, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> "Brendan Jurd" <direvus@gmail.com> 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
