Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:96680
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.161.173 as permitted sender)
MIME-Version: 1.0
Sender: morrison.levi@gmail.com
In-Reply-To: <5219c048-66f1-be9a-ae3c-15633a65e263@garfieldtech.com>
References: <44244cfd-c8ad-e0a9-4e70-380a8a8c940c@syberisle.net>
 <CAFMT4Nr+d1k20aY0P9o1G9KU_C9wz2cg2UOPrbE3qbXDBhqe3w@mail.gmail.com>
 <CAF+90c_7g_YdxTfUX8t_OifVJTL=OcvjsC_smWc95M-CcRYaNw@mail.gmail.com>
 <CADqTB_jN6RQxEP5tAJY4T+kuv_rkTzRNvL9T1j-iVX5LG5H3Gg@mail.gmail.com>
 <CAF+90c_rE7DiJ8KEYzFO9GHqx56rin+CiByPe7hRF0fUm-zKBQ@mail.gmail.com>
 <01f2f3da-592b-fd23-709d-ad620422f56d@fleshgrinder.com> <7fb9a5d5-318d-7ed3-59eb-d9845ed5dec1@gmail.com>
 <b5877ebc-95c7-bc68-184d-c50cd6cd52ae@librelamp.com> <5219c048-66f1-be9a-ae3c-15633a65e263@garfieldtech.com>
Date: Mon, 31 Oct 2016 11:57:10 -0600
Message-ID: <CAFMT4NqAQHqGRuj41Fup+sFbMiX73Dtyf1kdJvVBPosENAGPcg@mail.gmail.com>
To: Larry Garfield <larry@garfieldtech.com>
Cc: internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] Allow Iterator to be used with current, next, reset,
 key functions
From: levim@php.net (Levi Morrison)

On Mon, Oct 31, 2016 at 11:27 AM, Larry Garfield <larry@garfieldtech.com> wrote:
> On 10/31/2016 11:26 AM, Alice Wonder wrote:
>>
>> On 10/30/2016 10:19 AM, Rowan Collins wrote:
>>
>>>
>>> - deprecate reset(), end(), each(), and key()
>>> - introduce array_first(), array_last(), array_first_key() and
>>> array_last_key()
>>> - document replacements for whatever other use cases we can find
>>> examples of
>>>
>>
>> As a user I certainly like those names better. Semantic function names are
>> a definite win in my view.
>
>
>
> Question: Now that we have "iterable" as a type, should we be adding more
> array-specific functions or should such operations be designed, and named,
> to operate on any iterable?  (first(), first_key(), etc.)

At least for last and last-key I do not think we should make them work
with any iterable. My reasoning is that we do not want to consume the
iterator for them because many iterators are not rewindable (such as
generators). The first and first-key routines should generally work
because `rewind(); valid(); current(); rewind();` doesn't contain a
`next()` call.

Rewinding without iterating works for generators: https://3v4l.org/O7ZKO
Rewinding after a next() call: https://3v4l.org/QI434