[wp-trac] [WordPress Trac] #16958: Dead Code in WP_Http::processResponse()

[WordPress Trac]

#16958: Dead Code in WP_Http::processResponse()
--------------------------+------------------------------
 Reporter:  hakre         |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  lowest        |   Milestone:  Awaiting Review
Component:  HTTP          |     Version:  3.1
 Severity:  trivial       |  Resolution:
 Keywords:  has-patch     |
--------------------------+------------------------------

Comment (by hakre):

 Replying to [comment:1 dd32]:
 > Is this an actual bug? http://wpdevel.wordpress.com/2011/03/23/code-
 refactoring/

 I rate dead code as a bug. Bugs have not much in common with the issue
 about applying coding standards you referenced.

 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].

 I 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.

 So maybe we should turn this into a collecting ticket a core dev can
 review each some days.

 Related: #16978, #16942

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