[wp-trac] [WordPress Trac] #17817: do_action/apply_filters/etc. recursion on same filter kills underlying call

[WordPress Trac]

#17817: do_action/apply_filters/etc. recursion on same filter kills underlying call
-------------------------------------------------+-------------------------
 Reporter:  kernfel                              |       Owner:
     Type:  defect (bug)                         |      Status:  reopened
 Priority:  normal                               |   Milestone:  Future
Component:  Plugins                              |  Release
 Severity:  normal                               |     Version:  2.2
 Keywords:  dev-feedback has-unit-tests has-     |  Resolution:
  patch                                          |     Focuses:
-------------------------------------------------+-------------------------

Comment (by jbrinley):

 Replying to [comment:72 leewillis77]:
 > At least we'd need to verify that creating a copy of the args rather
 than reusing a reference doesn't impact on performance (CPU or memory)

 In most cases, the difference was inconsequential. I had to bump the
 number of arguments up to around 100 before I started seeing a noticeable
 (i.e., > 5%) impact on performance. 17817.9.patch negates that impact, by
 avoiding the copy entirely (there will still be a copy when the callback
 is called, but there won't be two copies made).

--
Ticket URL: <https://core.trac.wordpress.org/ticket/17817#comment:77>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform