Status

()

Core
ImageLib
--
enhancement
VERIFIED WONTFIX
16 years ago
13 years ago

People

(Reporter: tor, Assigned: tor)

Tracking

Trunk
PowerPC
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

2.58 KB, patch
Biesinger
: review+
blizzard
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

16 years ago
Remove unused RGBA code, let libpng do the word padding for MacOS.
(Assignee)

Comment 1

16 years ago
Created attachment 90331 [details] [diff] [review]
patch
(Assignee)

Updated

16 years ago
OS: Linux → MacOS X
Hardware: PC → Macintosh
(Assignee)

Comment 2

15 years ago
Created attachment 120310 [details] [diff] [review]
merge to tip
(Assignee)

Updated

15 years ago
Attachment #90331 - Attachment is obsolete: true
(Assignee)

Updated

15 years ago
Attachment #120310 - Flags: review?(cbiesinger)
Attachment #120310 - Flags: review?(cbiesinger) → review+
(Assignee)

Updated

15 years ago
Attachment #120310 - Flags: superreview?(blizzard)
Comment on attachment 120310 [details] [diff] [review]
merge to tip

sr=blizzard
Attachment #120310 - Flags: superreview?(blizzard) → superreview+
(Assignee)

Comment 4

15 years ago
Checked in.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED

Comment 5

15 years ago
Please reopen.  The checkin triggered bug #202376 and its duplicates.

Glenn
(Assignee)

Comment 6

15 years ago
Backed out.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Updated

15 years ago
Severity: normal → enhancement

Comment 7

15 years ago
Red is full-range but appears to be inverted--at least approximately; some of
the yellow areas appear darker than they should, but that may be due to gamma
correction.

Both green and blue are compressed into the upper quarter of the range (191+ and
188+, respectively), and both appear to be folded or wrapped--for example, the
chicken should be nearly black, but only its edges are darker, while the middle
is as bright as the wall above it.  The main kitten is darkest in its
medium-level parts, equally light in both its lightest and darkest regions.

I don't see anything immediately obvious in the patch that could do that, but as
noted in the other bug, the set-filler call will boost the reported number of
channels from 3 to 4, which presumably screws up the logic elsewhere.

Two other things come to mind:  despite the code it replaces, might it be
worthwhile to set the filler value to 0xff so that it can be used as alpha (if
only to test an alternate code path)?  And is decoder->colorLine ever allocated
if channels is _not_ greater than 3? (Maybe it doesn't need to be...)  I'm
looking only at the patch itself, not the full source code, so I could well be
missing things.  (Was it the intent to remove the RGBA/BGRA code path even for
non-Mac builds?)

(Assignee)

Comment 8

15 years ago
png_set_filler() wasn't doing what I expected and will cause the interlace
buffer to bloat.  Closing as WONTFIX.
Status: REOPENED → RESOLVED
Last Resolved: 15 years ago15 years ago
Resolution: --- → WONTFIX

Updated

13 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.