Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:114982
To: Marco Pivetta <ocramius@gmail.com>,
 PHP Internals List <internals@lists.php.net>
References: <CADyq6sJX0BbiSBek9dAvgTtPMCbM-nObRBGtu27buqeMt8FWgA@mail.gmail.com>
 <CADyq6s+_CfVFE8rcrG2foMmc12=PXDkMYtwJUmf8iY+ezZ672Q@mail.gmail.com>
Message-ID: <30330665-bac3-b9df-b928-9d5272c2a00e@gmx.de>
Date: Mon, 21 Jun 2021 19:08:50 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.11.0
MIME-Version: 1.0
In-Reply-To: <CADyq6s+_CfVFE8rcrG2foMmc12=PXDkMYtwJUmf8iY+ezZ672Q@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Language: de-DE
Content-Transfer-Encoding: quoted-printable
Subject: Re: [RFC] make Reflection*#setAccessible() no-op
From: cmbecker69@gmx.de ("Christoph M. Becker")

On 21.06.2021 at 17:53, Marco Pivetta wrote:

> On Sun, Jun 13, 2021 at 6:44 PM Marco Pivetta <ocramius@gmail.com> wrote=
:
>
>> I'm posting here to introduce a new simplification, as well as
>> quality-of-life-improving RFC:
>>
>> https://wiki.php.net/rfc/make-reflection-setaccessible-no-op
>>
>> The RFC is quite minimal, and proposes removing any runtime behavior fr=
om
>> `ReflectionMethod#setAccessible()` and
>> `ReflectionProperty#setAccessible()`, making `ReflectionMethod` and
>> `ReflectionProperty` accessible by default.
>
> Since feedback has been relatively minimal and positive, I will likely
> start voting tomorrow, since I don't want trouble with any upcoming 8.1
> deadline (plus I'll otherwise just forget).

Given that private constants and static properties already can be
accessed right away, I'm in favor of this RFC.

=2D-
Christoph M. Becker