Closed
Bug 544747
Opened 15 years ago
Closed 15 years ago
Update libpng to version 1.4.1
Categories
(Core :: Graphics: ImageLib, defect)
Core
Graphics: ImageLib
Tracking
()
VERIFIED
FIXED
People
(Reporter: glennrp+bmo, Assigned: glennrp+bmo)
References
Details
Attachments
(1 file, 1 obsolete file)
162.37 KB,
patch
|
joe
:
review+
|
Details | Diff | Splinter Review |
I expect to release libpng version 1.4.1 on or about February 18th. It offers increased speed and reduced memory usage compared to the current embedded libpng versions.
Assignee | ||
Comment 1•15 years ago
|
||
Assignee | ||
Comment 2•15 years ago
|
||
The Tryserver results for the v01 patch are at
http://build.mozilla.org/tryserver-builds/glennrp@gmail.com-v01n-1.9.3-Feb08-libpng-1.4.1
Comment 3•15 years ago
|
||
did all tests, reftest and otherwise continue to pass after that try run?
Assignee | ||
Comment 4•15 years ago
|
||
Yes, I received 9 messages from the Tryserver, reporting success on all builds and unittests.
Comment 5•15 years ago
|
||
All arrays are local now, right? There are still some leftovers:
#ifdef PNG_USE_LOCAL_ARRAYS
#endif
Also, this line at the end of pngrutil.c is not needed:
png_uint_32 row_bytes;
Assignee | ||
Comment 6•15 years ago
|
||
Libpng-1.4.1 is out. Here is the patch, and successful Tryserver builds are here:
http://build.mozilla.org/tryserver-builds/glennrp@gmail.com-v02c-1.9.3-libpng-1.4.1-Feb26
Attachment #425817 -
Attachment is obsolete: true
Attachment #429288 -
Flags: review?(joe)
Comment 7•15 years ago
|
||
Comment on attachment 429288 [details] [diff] [review]
Update libpng in trunk to version 1.4.1 (v2)
Let's get this into mozilla-central!
Attachment #429288 -
Flags: review?(joe) → review+
Comment 8•15 years ago
|
||
Group: mozilla-confidential
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•15 years ago
|
||
Now that 1.4.1 is checked in we should require the system library to be at version 1.4.1 as well. This would allow some simplification of nsPNGDecoder.cpp. I'm spinning off bug #551438 for that.
Status: RESOLVED → VERIFIED
Updated•15 years ago
|
Group: mozilla-confidential
You need to log in
before you can comment on or make changes to this bug.
Description
•