Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:124768
To: Larry Garfield <larry@garfieldtech.com>, php internals
	<internals@lists.php.net>
Subject: Re: [PHP-DEV] [Vote] Asymmetric visibility v2
Thread-Topic: [PHP-DEV] [Vote] Asymmetric visibility v2
Thread-Index: AQHa34lRjyqwNE11uUCnDWlPKq5s57IYuPpz
Date: Mon, 5 Aug 2024 13:49:25 +0000
Message-ID:
 <PH7PR16MB5781388723D03C148745639CDEBE2@PH7PR16MB5781.namprd16.prod.outlook.com>
References: <bf5f3563-46a8-46d0-bcce-307272e1c5fb@app.fastmail.com>
In-Reply-To: <bf5f3563-46a8-46d0-bcce-307272e1c5fb@app.fastmail.com>
Accept-Language: en-US
Content-Language: en-US
msip_labels:
x-ms-exchange-messagesentrepresentingtype: 1
x-tmn: [YmRvCwaeaGckCBHThg897DHHfnZyS3lJqi37Al9boMs=]
x-ms-publictraffictype: Email
x-ms-traffictypediagnostic: PH7PR16MB5781:EE_|SA1PR16MB4777:EE_
x-ms-office365-filtering-correlation-id: dec80be3-59d2-4dc7-45b7-08dcb5556b07
x-microsoft-antispam:
 BCL:0;ARA:14566002|15030799003|19110799003|461199028|8060799006|1602099012|4302099013|3412199025|440099028|102099032;
x-microsoft-antispam-message-info:
 Xmhn9KxThgY7SJomPzZlbun+ESnLfrHTg0WdzGIqTmQVq5iVIInolNq6KiF0F2puv9bxjSXSwSQ0AXWpiqGEbXHMhzvoxwqzesmINlEt4yV+IQ3vUJ+mobYa63oYYYdy33NuOeM2AcykkNaoGAJBPOLzkRLpJUM0+1qtwSoT6SrY1jqENmbBpDzF+tHmdfb0+QlOz902RQpYETztIuN5nIKV4WKYyHGS0MsjcnOO998STg1G6Fy1Jxcr6SWS2kl01aKgAf+JRB3WnqGuUhc0cyQZ2Z0PqGD71Hiae42/hDqrrGGL9Jcrnw9V8UDz7fJeUUHamHDaeSkE7BKaI4gmcaxRM4qrpezOrG//qZe71oOZBfbadpD799TaWD1yxUUeq4EBJkRU/hyGtohRNz2r8nc7gJZhiMdXksI7gkZUI+yi2/uusiORTEkhy9ka3vX0PeUDCyNRMm/2hjJZrkX9GP5/c2avSlKKQlKjilzghoynTuxj/s5zZyhr3PmZUMVV5LbTGWDHus6X8mnpTcjtVUofkJE4NS09WUtvn+SPsjZ6mHVxp023CcsaQrYSX3gjq6KTsmIMn1f2chMSUVbzEFfYzJuU9y1W8oPTukz6gDpV9b5mq+DKLqNOIyOGYPgosPGke/D2uqMQqpp5oad20p/Ws9kLkdOrZEudqReIbCb5qJ4wZwTHFopXumfXqbnGNl11WZRWn7Fy5hbzginD3ceVptUwQSk450LbYQ8ZH1Qb5+Ni91oEwghRmbNnMKPyNgOLGNEhAHGqKhpB+6PpDrjf0YJMwy8lKulgiPsArweyd85WLHIMBiuV0qCu1kIz6fPL9esOD5HxPXTU3H1JwQ==
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?iso-8859-1?Q?VIZ4+iUCfLR5mS8mi2m7Hz8YBuRs9yhb7HNjXKeX+INMFgdA6ebaOhG75a?=
 =?iso-8859-1?Q?rudFPxBh+JeO4f+6faWcuThexwgnh2mjh5V6YVLrOsyjGh0aWeui1bml5e?=
 =?iso-8859-1?Q?1HCRewUoNgvpFOGNujNNapT7e86ym7cRU9mEwh/bMgxGpJDikb1USQGiyn?=
 =?iso-8859-1?Q?ImwUF5oDs0njGLGhXVBOBfPLWiFfiZFuI28NMAa3YPn2F/cl/Do+M353AC?=
 =?iso-8859-1?Q?1Z3ZZIVnfJJLAixxk/PKzSxz2AYi5Apj42Epnyt7YK7jLDcSkZmKDvRky5?=
 =?iso-8859-1?Q?3wE2mHt2qqmCPtiGXIUJkbzrPBzrtDXIdeYUHNZIGr88HpJjnTSSeTMX45?=
 =?iso-8859-1?Q?X8wbsC886C3Aki08fVRszlZwjh2wG2n8gmn54juj3lw+23NH72ZOdwi2St?=
 =?iso-8859-1?Q?JNI9Sz7wk/tMxCJBkapr5P3tcCDaeJxEaRoKsgLWcu7mF58HKaFo2b6KQf?=
 =?iso-8859-1?Q?Xtlm/tPHzlOu4B84qmfeV7YqzI5x6adij1RnRw2yoP4hTEP2WNrABrxEqA?=
 =?iso-8859-1?Q?R2yIpO45hUTfC9U8MttjOeJvTqGEBkoz2V7kuSi6sX5i+8rOzH1SYH6/3Y?=
 =?iso-8859-1?Q?cTkKjyXuVEkBQBeM4VTmHaG1C3VD8Aess9GmBFFd+HoqSSGhbw9Gj3fpEO?=
 =?iso-8859-1?Q?TW9MA+dGeKkSaaJ5Ly0bSxpisN71+adMP7WASSEatlX+oiC4VGljKbwa/G?=
 =?iso-8859-1?Q?3r+of+8Ao2vuBkHjkpIWX8WkG3fAirN10pV9ookKIXNx/qA7V7nRpNMaHB?=
 =?iso-8859-1?Q?XGZRN50hcfJpN2LfBY8yOGnQIFeK3RnhImSrOPaydMV8KDCti3cvPRnL/6?=
 =?iso-8859-1?Q?O26Trzk/jbYKYH+/glefNCx5ImvwGzWY3jxSDr2EPkBJgGP/imCYbQ4cnm?=
 =?iso-8859-1?Q?fbHkQdnYXC9zzEm/2MRS7m0loPFo/GkQuApNqV39adwxAcRGzcpyZRSpx4?=
 =?iso-8859-1?Q?TSAhxgp83MgyHsHWuOo2CKX3uw353Yw6Rvo/5Pw5xyqdhIYKGJ/g9vhQKY?=
 =?iso-8859-1?Q?oH/lnPQ8/rnQO8G2w/vWP8LRoUVXSagD1g1RchqLQHzHf+rwzlEjWBECa0?=
 =?iso-8859-1?Q?kU71s42AQS3xfbW0c0RCSYi8k4GwxXGrjV/F4Qou5KpwKnOY32Lc+tDksi?=
 =?iso-8859-1?Q?HRjw+xmSaHSn9y4LvTWMpkYdhxAM9RbF7fQ0EUreApoHX9vdwZj0gG8SZz?=
 =?iso-8859-1?Q?Rttsot/29EuJODpmGQXbzBg84Q+ONHgq3/x1n2jIExC6XL28osFXg9Za0z?=
 =?iso-8859-1?Q?PHbkn+wuNXJix6fbKng3TkvWgjGXPszIZdopfv6lc=3D?=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
list-help: <mailto:internals+help@lists.php.net
list-unsubscribe: <mailto:internals+unsubscribe@lists.php.net>
list-post: <mailto:internals@lists.php.net>
List-Id: internals.lists.php.net
x-ms-reactions: disallow
MIME-Version: 1.0
X-OriginatorOrg: outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PH7PR16MB5781.namprd16.prod.outlook.com
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-CrossTenant-Network-Message-Id: dec80be3-59d2-4dc7-45b7-08dcb5556b07
X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Aug 2024 13:49:25.2129
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR16MB4777
From: theodorejb@outlook.com (Theodore Brown)

On Fri, July 26, 2024 at 12:25 Larry Garfield wrote:=0A=
=0A=
> Voting for Asymmetric Visibility is now open.=0A=
>=0A=
> https://wiki.php.net/rfc/asymmetric-visibility-v2=0A=
=0A=
Hi Larry and Ilija,=0A=
=0A=
Thank you for all your work on this RFC!=0A=
=0A=
One part that doesn't make sense to me is this sentence near the end in the=
 "Readonly is incompatible with inheritance" section:=0A=
=0A=
> With asymmetric visibility, the `readonly` usage here can be replaced wit=
h `protected protected(set)` or `readonly protected protected(set)`, avoidi=
ng the need to double-declare properties.=0A=
=0A=
`protected protected(set)` just removes readonly, and is the same as `prote=
cted`, right? So couldn't such a change be done now without asymmetric visi=
bility?=0A=
=0A=
And the change to `protected protected(set) readonly` as shown in the examp=
le also seems unnecessary, since earlier in the RFC it says that readonly w=
ill be changed to imply `protected(set)` rather than `private(set)`. So it =
seems that no change is needed to the abstract class here - the properties =
can remain `protected readonly`, and the implementation can be simplified s=
ince readonly properties will now imply `protected(set)`.=0A=
=0A=
Or have I misunderstood something here?=0A=
=0A=
Thanks,=0A=
Theodore=