Re: [GENERAL] Empty arrays with ARRAY[]

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