Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:96732
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.161.169 as permitted sender)
MIME-Version: 1.0
Sender: morrison.levi@gmail.com
In-Reply-To: <3fcfd89c-12d8-8fa9-eab7-5461771fdb9c@syberisle.net>
References: <44244cfd-c8ad-e0a9-4e70-380a8a8c940c@syberisle.net>
 <eb933424-cafa-c5b7-ec20-254d52d62a7b@syberisle.net> <CAFMT4NoJMrn9EAgdvJWprT68TW0wS+y8-1vcWRgNsiV9N=Zt6w@mail.gmail.com>
 <3fcfd89c-12d8-8fa9-eab7-5461771fdb9c@syberisle.net>
Date: Fri, 4 Nov 2016 09:10:58 -0600
Message-ID: <CAFMT4NoO82GKhXGrMFpoYr1-yMjmTY+kQAoxnWwHQDbP9-9OKg@mail.gmail.com>
To: David Lundgren <dlundgren@syberisle.net>
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 Thu, Nov 3, 2016 at 8:52 PM, David Lundgren <dlundgren@syberisle.net> wrote:
> On 11/3/16 2:58 PM, Levi Morrison wrote:
>>> I think it would be bad to remove these functions and force users to
>>> wrap arrays with ArrayIterator, just to manually iterate an array. After
>>> spending a little time looking through how these functions are being
>>> used in frameworks and libraries, removing them seems like it will cause
>>> more harm than good, even in a future version of the language.
>>
>> Can you show some examples of frameworks and libraries you found where
>> it would cause more harm than good?
>
> Apologies, that was a subjective statement. I also made it appear that
> the usage of these functions was greater than it may really be, and
> would cripple said projects, that was not my intention, and most
> definitely not the case.
>
> These are the ones I remember, and another way could be used if
> current/next/key were not available for these contexts:
>
> Yii CMarkdownParser: https://git.io/vXCiy
> ZF2 stdlib FastPriorityQueue: https://git.io/vXCiH
> Drupal DI Container: https://git.io/vXCPB
> Symfony MO file loader: https://git.io/vXCPK
> Symfony HttpFoundation Request: https://git.io/vXCXW
>
>
> Dave
> --
> David Lundgren
> dlundgren@syberisle.net
> GPG: 0x26F54D7F
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php

Many of these just seem to use `current` to get the first item and
would easily migrate to a new API such as `first()`.