[wp-trac] [WordPress Trac] #53858: PHP 8.1: syntax error due to new 'readonly' property
WordPress Trac
noreply at wordpress.org
Wed Aug 4 01:05:30 UTC 2021
#53858: PHP 8.1: syntax error due to new 'readonly' property
--------------------------------------------+---------------------
Reporter: haosun | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: 5.9
Component: General | Version:
Severity: normal | Resolution:
Keywords: php81 needs-dev-note has-patch | Focuses:
--------------------------------------------+---------------------
Comment (by haosun):
Replying to [comment:19 jrf]:
> @haosun Regarding the performance impact: fatal errors trump performance
considerations any time. Including an extra file will always have a small
performance impact, but can't be avoided in this case as we do need to
mitigate the BC-break as much as possible.
Thanks for your explanation. I further tested under more machines with
different CPUs and found that there is only negligible performance impact.
@ayeshrajans Thanks a lot for your previous patches.
Could you help update the patch with the suggestions from @jrf and
@SergeyBiryukov?
--
Ticket URL: <https://core.trac.wordpress.org/ticket/53858#comment:23>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list