Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:110990
MIME-Version: 1.0
References: <CAP=4hCKy+Vzr_GgpCTNcEC6YEnRVS4pXPyd+_DLYrUA8Uz5yrA@mail.gmail.com>
In-Reply-To: <CAP=4hCKy+Vzr_GgpCTNcEC6YEnRVS4pXPyd+_DLYrUA8Uz5yrA@mail.gmail.com>
Date: Mon, 13 Jul 2020 22:44:45 -0300
Message-ID: <CAOsHV+v-XBDtM=5UMVC2F1uEG=m2c+6f=G-5mZbtcEz+teshtg@mail.gmail.com>
To: Gabriel Caruso <carusogabriel@php.net>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000600a8605aa5cf4b9"
Subject: Re: [PHP-DEV] Drop warning about non-public magic methods
From: marcio.web2@gmail.com (Marcio Almada)

--000000000000600a8605aa5cf4b9
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Grabriel,

Hello internals,
>
> During the development of https://wiki.php.net/rfc/magic-methods-signatur=
e
> I
> found something that I'd like to share with you, and possibly propose to
> drop it: there's a warning that php-src throws when a magic method is
> declared with visibility different than `public`: https://3v4l.org/LKnKB.
>
> This warning does not make much sense as the magic method is executed
> regardless of its visibility. Should it be dropped?
>
> - Gabriel Caruso
>

Besides the BC break involved is there any other constraint
not to emit a full compile error given that the engine still
executes the method call and the visibility, other than public,
is therefore invalid code?

Ty,
M=C3=A1rcio

--000000000000600a8605aa5cf4b9--