Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:110136
MIME-Version: 1.0
References: <CAF+90c8BeG9GwQOVypcNe3p5Yp9LefsWHfH5Wrjq1o6HZvrZVQ@mail.gmail.com>
 <CAF+90c_mfx3fbwWGusKH+LdXcMV7g-EiX5H15-AAWmhPSxnLAA@mail.gmail.com>
In-Reply-To: <CAF+90c_mfx3fbwWGusKH+LdXcMV7g-EiX5H15-AAWmhPSxnLAA@mail.gmail.com>
Date: Tue, 12 May 2020 07:49:09 -0500
Message-ID: <CAESVnVq7ZeJvb5M9TNFpa6_=YmuFMxmReW5BGa_cNMTbziV1sg@mail.gmail.com>
To: Nikita Popov <nikita.ppv@gmail.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000068628305a572e448"
Subject: Re: [PHP-DEV] Re: Making all Traversables an Iterator or IteratorAggregate
From: pollita@php.net (Sara Golemon)

--00000000000068628305a572e448
Content-Type: text/plain; charset="UTF-8"

On Tue, May 12, 2020 at 3:26 AM Nikita Popov <nikita.ppv@gmail.com> wrote:
> // WeakMap::getIterator(): Iterator
> ZEND_METHOD(WeakMap, getIterator)
> {
>     if (zend_parse_parameters_none() == FAILURE) {
>         return;
>     }
>     zend_create_internal_iterator_zval(return_value, ZEND_THIS);
> }
>
Given that the body of this method seems to usually (always?) be the same,
why not define it in InternalIterator and allow it to be inherited?

> There's some bikeshed potential here regarding the class name.
>
Not personally over-picky, but I do agree that "Internal*" is a bit
awkward.  Unfortunately there's not much that's better and appropriate for
exposing to scripts.  This might be one of those rare occasions where
exposing "Zend" into userspace makes sense.  "PHP" is overloaded into
several meanings that are significant for userspace developers, but
something like "ZendIterator" might convey the right level of "This has to
do with the engine, please move along".  Or maybe go verbose:
'IteratorForExtensionClassImplementations'. :p

> ZEND_ASSERT(scope->get_iterator != zend_user_it_get_new_iterator); >
Does this mean that if I do `class Foo implements InternalIterator {}` in a
script, I can assert (crash) PHP? I don't see anything obvious (at a
glance) preventing me from inheriting from InternalIterator.

-Sara

--00000000000068628305a572e448--