Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:119379
MIME-Version: 1.0
References: <CAH5C8xUd4ueJ62pR6vT2haz9rFgrgspJQui1rPCqTneupD+tsQ@mail.gmail.com>
 <CADyq6sLFZpM1A+V-aa3dWjY1mL=65e-kxeiyBHABNPZR2GfpKQ@mail.gmail.com>
 <CAFPFaMKc-PQD8YD6yDWPrMONcJdYqFCXgyJZuZZBVsT7VSy89A@mail.gmail.com>
 <CADyq6sJxNQKvDNsT48tsG=AUS8snbGvuR332Yaip-krd+yynaQ@mail.gmail.com>
 <CAFPFaML4Xq7Oo+pN7cAj3aQt3y3et33EzsQhxhqNZA22nOUuNw@mail.gmail.com>
 <CADK1yXLSVH0X=dteA=GO+=UPSzSDfCecALytjRFG-u+0QRyGWg@mail.gmail.com>
 <a719c9d1-de30-41d2-b7ae-24dc53a57d90@app.fastmail.com> <CAH5C8xUer=mrRNYrdYpiYk6Z+nfaeZuacCJdAa3Oi8hyw97+Cg@mail.gmail.com>
 <2115758d-79e9-340e-ed7b-176dd0927c79@bastelstu.be>
In-Reply-To: <2115758d-79e9-340e-ed7b-176dd0927c79@bastelstu.be>
Date: Fri, 20 Jan 2023 17:54:03 +0100
Message-ID: <CAH5C8xWi=a+9VsSBFC4iPGvnHEBEG8Gm84+Co9+Kb491GrAukA@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="0000000000004b7a9205f2b4e6a2"
Subject: Re: [PHP-DEV] [RFC] [Discussion] Readonly class amendments
From: kocsismate90@gmail.com (=?UTF-8?B?TcOhdMOpIEtvY3Npcw==?=)

--0000000000004b7a9205f2b4e6a2
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Claude, Tim,

One shortcoming around readonly classes that I just figured out, is that it
> is not possible to use them as anonymous class:
>

Nice catch! As you wrote, it's indeed an oversight of the readonly class
implementation. Fortunately, we already have a PR with the fix. :)

I'm confused by the linked PRs for the implementation, because they do
> not appear to match what's proposed. Specifically the one for proposal
> #2 (allow modification in __clone) appears to include unrelated changes
> (a clone-with{} syntax).
>

Sorry for the confusion! The PR contains an implementation for the "clone
with" indeed
just because it builds on top of some specifics of the 2nd proposal in the
"Readonly amendments" RFC. However,
the first few (4) commits are related to allowing modification of readonly
properties in __clone().

M=C3=A1t=C3=A9 Kocsis

--0000000000004b7a9205f2b4e6a2--