WordPress.org

Make WordPress Core

Changes between Version 1 and Version 2 of Ticket #16958, comment 2


Ignore:
Timestamp:
03/29/11 09:52:57 (3 years ago)
Author:
hakre
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #16958, comment 2

    v1 v2  
    22> Is this an actual bug? http://wpdevel.wordpress.com/2011/03/23/code-refactoring/ 
    33 
    4 I rate dead code as a bug. Bugs have not much in common with the issue about applying coding standards you referenced. 
     4I rate dead code as a bug if the code in question does not give any enhance but instead only consumes. Bugs have not much in common with the issue about applying coding standards you referenced. 
    55 
    6 I think once reported it is often easy to fix and make the code [http://wpdevel.wordpress.com/2011/03/18/wordpress-3-2-the-plan-faster-lighter/ lighter]. 
     6I think once reported it is often easy to fix and make the code [http://wpdevel.wordpress.com/2011/03/18/wordpress-3-2-the-plan-faster-lighter/ lighter]. But it's not that I report them out of the blue. 
    77 
    88I normally spot those while reviewing code in other tickets, so I think those are worth to report.  Maybe we can have a family ticket for those. It's just like how I find those: [attachment:ticket:16978:16978.patch 16978.patch] which for example already has been committed. It's just not that always there is a ticket already to add the patch to.