Closed Bug 702555 Opened 8 years ago Closed 7 years ago

debug method PrintImageDecoders() in imgLoader should be removed (Doesn't work and is confusing)

Categories

(Core :: ImageLib, defect, trivial)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: fowl, Unassigned)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0a1) Gecko/20111114 Firefox/11.0a1
Build ID: 20111114031820

Steps to reproduce:

http://hg.mozilla.org/mozilla-central/file/9ae1d4f44b8b/image/src/imgLoader.cpp#l108

While playing around with writing a toy image decoder (in js) this code kept showing up in searches and confusing me.  It's been horribly broken since bug 513681 and it's been there since pre-hg.
Severity: normal → trivial
Component: General → ImageLib
OS: Windows 7 → All
Hardware: x86_64 → All
This is trivial really, but I did build and run it on my machine anyway. 

I used getReviewer.py to find a reviewer, so if it's not you apologies but I'm just following the instructions :P
Attachment #654600 - Flags: review?(justin.lebar+bug)
> I used getReviewer.py to find a reviewer, so if it's not you apologies but I'm just 
> following the instructions :P

I'm a fine reviewer for this code.  But also, choosing a poor reviewer is not a faux pas; the reviewer should redirect your request towards someone else without complaint.  :)

Thanks for the patch!
Attachment #654600 - Flags: review?(justin.lebar+bug) → review+
Normally we'd ask for a patch formatted for checkin-needed [1] before pushing, but since this is your first one, I've added that info and pushed:

  https://hg.mozilla.org/integration/mozilla-inbound/rev/16e1ff49a6a5

Thanks again for the patch!

[1] http://blog.bonardo.net/2010/06/22/so-youre-about-to-use-checkin-needed
https://hg.mozilla.org/mozilla-central/rev/16e1ff49a6a5
Status: UNCONFIRMED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.