Closed
Bug 1116746
Opened 9 years ago
Closed 9 years ago
Remove unused statistics from RasterImage
Categories
(Core :: Graphics: ImageLib, defect)
Core
Graphics: ImageLib
Tracking
()
RESOLVED
FIXED
mozilla37
People
(Reporter: seth, Assigned: seth)
References
Details
Attachments
(1 file, 2 obsolete files)
6.06 KB,
patch
|
tnikkel
:
review+
|
Details | Diff | Splinter Review |
There is some truly ancient statistics code in RasterImage that I don't think anyone has used in a very long time. (I checked around with people like bholley.) It would need to be rewritten as part of the multiple decoders for a single RasterImage refactoring (bug 1079627), but I don't think that's worth it since nobody uses it. Let's just rip it out.
Assignee | ||
Comment 2•9 years ago
|
||
I noticed that this change left |get_header_str| unused. In this new version I remove that too.
Attachment #8543107 -
Flags: review?(tnikkel)
Assignee | ||
Updated•9 years ago
|
Attachment #8542942 -
Attachment is obsolete: true
Attachment #8542942 -
Flags: review?(tnikkel)
Assignee | ||
Comment 3•9 years ago
|
||
I take it back; get_header_str is needed until a later patch in my queue removes some other code. Restored it here.
Attachment #8543454 -
Flags: review?(tnikkel)
Assignee | ||
Updated•9 years ago
|
Attachment #8543107 -
Attachment is obsolete: true
Attachment #8543107 -
Flags: review?(tnikkel)
Assignee | ||
Comment 4•9 years ago
|
||
Try job here: https://tbpl.mozilla.org/?tree=Try&rev=e75646f7143d
Updated•9 years ago
|
Attachment #8543454 -
Flags: review?(tnikkel) → review+
Assignee | ||
Comment 5•9 years ago
|
||
Pretty sure those oranges are all from other things in my patch queue. I went ahead and pushed: https://hg.mozilla.org/integration/mozilla-inbound/rev/25d6527235ca
https://hg.mozilla.org/mozilla-central/rev/25d6527235ca
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in
before you can comment on or make changes to this bug.
Description
•