Closed Bug 832487 Opened 8 years ago Closed 8 years ago
Update libpng to version 1
Libpng-1.5.14 is expected to be released on January 24th, 2013. Right now it's at version 1.5.14rc03. It would be good to test with rc03 to see if there are any problems, then land 1.5.14 after it's released.
Assignee: nobody → glennrp+bmo
Status: NEW → ASSIGNED
This is for testing only. Someone please run this through try-server
I don't see any makefile changes here to actually build the new ARM code?
(In reply to Ryan VanderMeulen from comment #2) > I don't see any makefile changes here to actually build the new ARM code? There aren't. We can revise the makefile/configure/mozpngconf.h whatever to enable ARM support in bug #832390 once we know libpng-1.5.14 is OK.
May I suggest a minor change in pngwutil.c (line 1049): - png_write_chunk_header(png_ptr, png_fdAT, 4 + length); + png_write_chunk_header(png_ptr, png_fdAT, (png_uint_32)(4 + length)); I see that's how it is used in other places: png_write_chunk_header(png_ptr, png_PLTE, (png_uint_32)(num_pal * 3)); png_write_chunk_header(png_ptr, png_pCAL, (png_uint_32)total_len); png_write_chunk_header(png_ptr, png_hIST, (png_uint_32)(num_hist * 2));
I think png.c is missing from your patch, Glenn... One more minor thing: last time we accidentally duplicated a comment in pngwrite.c It's the same comment at lines 317 and 323. We should remove the first one.
Attachment #704127 - Attachment is obsolete: true
applying 832487 unable to find 'media/libpng/arm/arm_init.c' for patching 1 out of 1 hunks FAILED -- saving rejects to file media/libpng/arm/arm_init.c.rej unable to find 'media/libpng/arm/filter_neon.S' for patching 1 out of 1 hunks FAILED -- saving rejects to file media/libpng/arm/filter_neon.S.rej patch failed, unable to continue (try -v) patch failed, rejects left in working dir Looks like you lost the hg add in there.
Add arm directory with 2 files
Attachment #704266 - Attachment is obsolete: true
patch for actual libpng-1.5.14, includes new apng.patch
Attachment #704288 - Attachment is obsolete: true
Comment on attachment 705742 [details] [diff] [review] v03 Update libpng to version 1.5.14 Looks good to me, thanks Glenn.
probably it make sense to move it to another bug, but not sure
Yes, we already have another bug for the ARM NEON stuff. It's bug #832390 which is shown as a dependency of this bug. Also, the "MOZ_" prefixes should be applied via macros in mozpngconf.h rather than changing them in the code.
Comment on attachment 705742 [details] [diff] [review] v03 Update libpng to version 1.5.14 Review of attachment 705742 [details] [diff] [review]: ----------------------------------------------------------------- r=try
Attachment #705742 - Flags: review?(joe) → review+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
5 years ago
You need to log in before you can comment on or make changes to this bug.