[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