Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:120595
MIME-Version: 1.0
References: <CAAKU0umNOJ+6x44WRuzc2e9_5sDehAq5UB_aFn+cT4wxTny9wg@mail.gmail.com>
 <CADK1yXKwdpHtZBbKZRHwfJUxJoM19LtjiyKOFvAiyCTYjc+xyA@mail.gmail.com> <cdc81eb6-886b-ab78-bc9b-832f0a71b27f@bastelstu.be>
In-Reply-To: <cdc81eb6-886b-ab78-bc9b-832f0a71b27f@bastelstu.be>
Reply-To: Levi Morrison <levi.morrison@datadoghq.com>
Date: Thu, 15 Jun 2023 09:52:28 -0600
Message-ID: <CAAKU0umb=PEFT9ZYLMpVgPO1X7KD=ZSN0pOH+3RHSM=JJBLcTw@mail.gmail.com>
To: =?UTF-8?Q?Tim_D=C3=BCsterhus?= <tim@bastelstu.be>
Cc: Deleu <deleugyn@gmail.com>, PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] [RFC] Interface Default Methods
From: internals@lists.php.net ("Levi Morrison via internals")

On Thu, Jun 15, 2023 at 9:16=E2=80=AFAM Tim D=C3=BCsterhus <tim@bastelstu.b=
e> wrote:
>
> Hi
>
> On 6/15/23 17:08, Deleu wrote:
> > The feature may introduce a new way for *Users of PHP* to break BC with
> > *Other Users of PHP*. The language change itself has no impact on PHP c=
ode
> > written prior to the feature. The additional note about how users may b=
reak
> > BC by using the feature would be a description of the feature itself, t=
hus
> > might be best declared as part of the Proposal instead.
> >
>
> There's also the possible impact with regard to static analysis tools,
> IDEs and formatters that do not expect a method body to exist within
> interfaces and thus might erroneously report such an interface as
> invalid. This might be considered an impact on code written prior to the
> feature existing.
>
> Best regards
> Tim D=C3=BCsterhus

I have added a brief section in RFC impact that code analysis tools
will need to be updated. This is implicitly true for many language
features, but I see no harm in calling it out in the RFC, so I have
done so. Thank you for the feedback.