Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:91122
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.52 as permitted sender)
To: internals@lists.php.net
References: <CAG2fJbZKSj2GMtXDbjpgR287SeqhutO6Ssk-ntPqx62Ux5Pnrg@mail.gmail.com>
 <CAF+u_BzAH35i06dE2fw3594vUfLfS8XvTQKRJVct-A186V1uEQ@mail.gmail.com>
Message-ID: <56B8C492.2060800@gmail.com>
Date: Mon, 8 Feb 2016 16:38:42 +0000
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101
 Thunderbird/38.5.1
MIME-Version: 1.0
In-Reply-To: <CAF+u_BzAH35i06dE2fw3594vUfLfS8XvTQKRJVct-A186V1uEQ@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Subject: Re: [PHP-DEV] Proposal for a new array function
From: rowan.collins@gmail.com (Rowan Collins)

Kinn Julião wrote on 08/02/2016 16:05:
> That's the nice thing about userland... you are giving them ability to
> implement what they want!
> `array_filter([...], 'is_int', ARRAY_FILTER_USE_KEY)` does the job as the
> others... don't think having a new function would prevent this.

It does the job, but it is much less efficient in some cases than the 
foreach and return false case, as it must examine every element; it also 
has unnecessary memory overhead, creating a filtered list which exists 
only to be counted and thrown away; and an internal implementation could 
be optimised even further by exploiting the way the engine stores the 
array internally.

PHP is at this stage expressive enough that a large number of core 
functions could be rewritten in multiple ways using other core 
functions, but that doesn't automatically rule out adding more functions 
to core where it is useful to do so.

Regards,
-- 
Rowan Collins
[IMSoP]