Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:120336
MIME-Version: 1.0
References: <e58dde83-654d-b0ad-de77-d0450a56ef75@bastelstu.be>
In-Reply-To: <e58dde83-654d-b0ad-de77-d0450a56ef75@bastelstu.be>
Date: Thu, 18 May 2023 10:12:28 +0200
Message-ID: <CADyq6sJ3aaipfagKtoqksOm2JFp-UN5H0hkrOndgKad2W9x3fg@mail.gmail.com>
To: =?UTF-8?Q?Tim_D=C3=BCsterhus?= <tim@bastelstu.be>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000002db1c405fbf35e79"
Subject: Re: [PHP-DEV] RFC [Discussion]: Marking overridden methods (#[\Override])
From: ocramius@gmail.com (Marco Pivetta)

--0000000000002db1c405fbf35e79
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hey Tim,

On Thu, 11 May 2023, 18:37 Tim D=C3=BCsterhus, <tim@bastelstu.be> wrote:

> Hi
>
> I'm now opening discussion for the RFC "Marking overridden methods
> (#[\Override])":
>
> ----
>
> RFC: Marking overridden methods (#[\Override])
> https://wiki.php.net/rfc/marking_overriden_methods
>
> Proof of concept implementation is in:
>
> https://github.com/php/php-src/pull/9836


The idea is neat, but the trend is roughly:

 * Many moving to `final` classes (finally!)
 * Moving to less methods in an `interface` (some RFCs around functional
interfaces, even)

I am not sure this RFC is really relevant: I can see some use in generated
code, since this would act as an engine-level checksum for some minimal
changes, but otherwise it's a bit bloated?

Would it perhaps make sense to have this in userland first, in phpstan or
psalm plugins, to see if there is interest?

--0000000000002db1c405fbf35e79--