Re: array_length()

Decibel! wrote:
> ISTM it'd be useful to have an array_length function (since I just wrote 
> one for work ;), so here's a patch. Note that I don't have the docs 
> toolchain setup, so I wasn't able to test the doc patches.

There is a tiny problem with this implementation: It returns null for an 
empty array, not zero.  This is because array_lower and/or array_upper 
return null for an empty array, which makes sense for those cases.  We 
could fix this by putting a coalesce around the expression, but since 
the array functions return null for all kinds of error cases, this might 
mask other problems.  Or we move to a C implementation.