[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