[wp-trac] [WordPress Trac] #17997: Code cleanup, minor code enhancements to "misc.php"

WordPress Trac wp-trac at lists.automattic.com
Tue Jul 5 18:51:28 UTC 2011


#17997: Code cleanup, minor code enhancements to "misc.php"
-------------------------+------------------------------
 Reporter:  Latz         |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  General      |     Version:
 Severity:  minor        |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------------------

Comment (by johnbillion):

 In [http://groups.google.com/group/wp-hackers/msg/87e3acf4c6d35f2b your
 post to wp-hackers] you stated:

 > I was changing spacing for a function anyway because it was horrible to
 read and to understand

 This is a good reason to refactor a particular piece of code. Refactoring
 the whole file, however, is not necessary. It should be avoided for the
 reasons pointed out in the WP Devel link above.

 Which function were you originally refactoring to improve the readability
 of? IMO you should resubmit a patch with just the refactoring you
 originally intended.

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


More information about the wp-trac mailing list