Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:102606
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain seld.be designates 74.125.82.50 as permitted sender)
To: internals@lists.php.net
References: <CAO7eTqNUo8Oy+D7PUO7acNL05EHiXdosRCZS2agiawu6JxV5+w@mail.gmail.com>
 <a4760a58-f46f-7577-bf29-fafeb1fbc40b@gmx.de>
 <CAO7eTqPZdxGO6E9Gb6Lq2UmghpBZE2i6Bh=zvc0Ok9-KNNtvSw@mail.gmail.com>
 <cb4e33be-0610-9bf5-cbb5-c412e7db71b4@gmx.de>
 <CAO7eTqNGp4gSEfaUK0hOquRz7o-ju183V_csdjiHkmGg5aMFEw@mail.gmail.com>
Message-ID: <00ee21cb-6a60-ea2d-c805-6ede99ff44b8@seld.be>
Date: Thu, 5 Jul 2018 16:43:02 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101
 Thunderbird/52.8.0
MIME-Version: 1.0
In-Reply-To: <CAO7eTqNGp4gSEfaUK0hOquRz7o-ju183V_csdjiHkmGg5aMFEw@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-US
Subject: Re: [PHP-DEV] Re: [RFC][Vote] iterable_to_array() and
 iterable_count()
From: j.boggiano@seld.be (Jordi Boggiano)

Hey,

Just to explain the No vote for iterable_count, I think we'd rather 
advocate people implement Countable in their iterables than to force the 
consumption of an iterator just to count it as this can have 
side-effects and render the iterator unusable. If Countable is 
implemented then count() will do the job and all is well. If consuming 
the iterator is desired then one still can do 
count(iterable_to_array($iter)) fairly easily.

Best,
Jordi


On 04/07/2018 00:44, Michael Moravec wrote:
> 2018-07-04 0:23 GMT+02:00 Christoph M. Becker <cmbecker69@gmx.de>:
>> Yeah, I'd prefer that.  It's not that 2018-07-17 12:00 UTC *couldn't*
>> work – it's just I'd rather have some buffer time (I presume there will
>> be more RFCs whose voting ends close to feature freeze).
> Alright, updated!
> RFC voting now closes on 2018-07-16 23:00 UTC.
>
> M.
>

-- 
Jordi Boggiano
@seldaek - https://seld.be