@Crell In the Property hooks RFC, just before the Proposal Summary heading, it says "With property hooks, this can be simplified to:" followed by a code example containing a private property. Is that example correct? I wouldn't expect the property to be externally writable unless it's public.
@Crell In the Abbreviated forms section, the second code block contains a private array $modified = []; property. What is it for? It doesn't appear to be referenced anywhere.
There's also a private bool $isModified = false; property in the first code block under Proposal Summary, which likewise doesn't appear to be referenced anywhere.
@Crell Just checking if you received this email: news-web.php.net/php.internals/122474. I believe those are actual problems, instead of just me complaining about syntax.
@TheodoreBrown Left over from an earlier draft, I think. Fixed.
@TheodoreBrown Hm, good catch. Added a write to it from the set hook.
@TimWolla 1 and 4 have been addressed. 2, I don't think the grammar is wrong? Where is it wrong? 3. Hm. We were mostly going for the "it's syntactically meaningless so don't do it" angle, but if final methods in final classes are allowed and ignored, it may make sense to follow suit. It doesn't really change the feature set at all.
@TheodoreBrown Also a leftover from earlier drafts. Thanks.
@TimWolla We've been discussing the enum option, and may yet go for it. I think the last time we discussed it we were leaning yes, but the question was whether getHooks() had any purpose at that point, or what it should do.
@Crell (2) I believe it might just be an 'and' missing before 'will'? I'd also insert an 'as' between 'hooks' and 'final', but that might just be my non-native-English brain.