Alex Hunsaker <badalex@gmail.com> writes:
> On Wed, Jul 11, 2012 at 9:54 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> I'm looking at this patch now. The restriction of array_remove to
>> one-dimensional arrays seems a bit annoying. I see the difficulty:
>> if the input is multi-dimensional then removing some elements could
>> lead to a non-rectangular array, which isn't supported. However,
>> that could be dealt with by decreeing that the *result* is
>> one-dimensional and of the necessary length, regardless of the
>> dimensionality of the input.
> Makes sense to me. +1
> The other option ISTM is to replace removed entries with NULL-- which
> I don't really like.
Well, you can do that with array_replace, so I don't see a need to
define array_remove that way.
regards, tom lane
