Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:114712
MIME-Version: 1.0
References: <CAH5C8xX87sVh5NDuQ1EfeEbSxdHawO5yNfk5zHEToAqYGJa7=Q@mail.gmail.com>
In-Reply-To: <CAH5C8xX87sVh5NDuQ1EfeEbSxdHawO5yNfk5zHEToAqYGJa7=Q@mail.gmail.com>
Date: Thu, 3 Jun 2021 16:54:55 +0200
Message-ID: <CAF+90c-cZBG2zpbQ=asU-yYEJdjpik5kKVE4KOpQ=KEP7yDD_Q@mail.gmail.com>
To: =?UTF-8?B?TcOhdMOpIEtvY3Npcw==?= <kocsismate90@gmail.com>
Cc: PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000010198905c3ddc3ef"
Subject: Re: [PHP-DEV] [RFC] [Vote] Adding return types to internal methods
From: nikita.ppv@gmail.com (Nikita Popov)

--00000000000010198905c3ddc3ef
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, Apr 22, 2021 at 9:42 AM M=C3=A1t=C3=A9 Kocsis <kocsismate90@gmail.c=
om> wrote:

> Hi Internals,
>
> I've just opened the vote about
> https://wiki.php.net/rfc/internal_method_return_types
> and I will close it on 2021-05-06.
>
> For prior discussion, please see https://externals.io/message/113413
>
> Regards:
> M=C3=A1t=C3=A9
>

One implication that I didn't fully appreciate is that this also applies to
interfaces. The migration to tentative return types hasn't finished yet,
but we already have

interface JsonSerializable {
    function jsonSerialize(): mixed;
}

Which requires implementers of that interface to also specify "mixed" or a
subtype of "array" in the implementation. I expect that there will also be
at least

interface Countable {
    function count(): int;
}

This feels a bit different than the class case, in that implementing
internal interfaces is common, while extending internal classes is mostly
an artifact of us not adding enough "final"s in the early days.

I'm okay with this, but wanted to point it out.

Regards,
Nikita

--00000000000010198905c3ddc3ef--