[wp-trac] [WordPress Trac] #25475: Hook Docs: wp-includes/cron.php

[WordPress Trac]

#25475: Hook Docs: wp-includes/cron.php
------------------------------+---------------------------
 Reporter:  tmtoy             |       Owner:  DrewAPicture
     Type:  defect (bug)      |      Status:  closed
 Priority:  normal            |   Milestone:  3.8
Component:  Inline Docs       |     Version:
 Severity:  normal            |  Resolution:  fixed
 Keywords:  has-patch commit  |
------------------------------+---------------------------

Comment (by nacin):

 add_query_arg() is a utility function but doesn't always need to be used.
 To eat the CPU cycles it brings, there should be some benefit to sanity
 (like escaping) and readability, but that wasn't the case here and I don't
 think it needed to change.

 Regardless, let's avoid these kinds of functional changes during inline
 documentation commits. Not a *huge* deal as things can always be reverted,
 but it makes it much harder to review the commits. Thanks!

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