[wp-trac] [WordPress Trac] #17111: Remove unnecessary copy-by-reference in do_action()

[WordPress Trac]

#17111: Remove unnecessary copy-by-reference in do_action()
-------------------------+------------------
 Reporter:  scribu       |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  3.2
Component:  Performance  |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------

Comment (by Denis-de-Bernardy):

 Replying to [comment:4 scribu]:
 > Link, or it didn't happen. :)

 Sorry, not in much of a drive to google.com/archive.org the reference.
 FWIW, though, I vaguely recall seeing as an example of good things to do
 in the codex somewhere between 2005 and 2006. Plus, in case memory serves
 me wrong, that was the only means of getting the expected behavior (with
 php4 objects) back then anyway.

 I'm arguably biased in some sense, since my own plugins called things that
 way back then. (Those versions are completely obsolete, so I don't care
 much; but pointing out why I'm raising the issue.)

 This probably may no longer be a necessity with php5 objects (since that
 is what the array with a reference was all about)... But the rational
 behind the original behavior shouldn't be forgotten. And I dare suggest
 that a few test cases might be needed before things do change.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/17111#comment:6>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software