From: Ross Burton <ross.burton@intel.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 2/2] cairo: drop transform patch, it's been rejected upstream
Date: Wed, 17 Apr 2013 14:34:28 +0100 [thread overview]
Message-ID: <1366205668-19096-2-git-send-email-ross.burton@intel.com> (raw)
In-Reply-To: <1366205668-19096-1-git-send-email-ross.burton@intel.com>
This fix was rejected by upstream, the correct fix is actually to fix libpng.
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/recipes-graphics/cairo/cairo/png.patch | 77 ++-------------------------
1 file changed, 3 insertions(+), 74 deletions(-)
diff --git a/meta/recipes-graphics/cairo/cairo/png.patch b/meta/recipes-graphics/cairo/cairo/png.patch
index 15ce80a..bf00332 100644
--- a/meta/recipes-graphics/cairo/cairo/png.patch
+++ b/meta/recipes-graphics/cairo/cairo/png.patch
@@ -1,81 +1,10 @@
-libpng 1.6 is stricter in various ways, which trips up Cairo's PNG loader.
+libpng 1.6 is stricter in various ways, which can trip up the PNG loader as it
+considers all warnings fatal.
-Upstream-Status: Submitted (first)/Backport (second)
+Upstream-Status: Backport
Signed-off-by: Ross Burton <ross.burton@intel.com>
-From 1535e4eeda7e0792fe5e7e5ab377c5253ee89ce7 Mon Sep 17 00:00:00 2001
-From: Ingmar Runge <ingmar@irsoft.de>
-Date: Tue, 16 Apr 2013 10:48:59 +0100
-Subject: [PATCH 1/2] png: fix transform ordering
-
-libpng 1.6 is stricter with the function ordering, emitting the warning "invalid
-before the PNG header has been read" when calling png_set_read_user_transform_fn
-whilst loading a PNG.
-
-So, re-order the functions to the order that libpng is happy with.
-
-Signed-off-by: Ross Burton <ross.burton@intel.com>
----
- src/cairo-png.c | 21 +++++++++++++++++++--
- 1 file changed, 19 insertions(+), 2 deletions(-)
-
-diff --git a/src/cairo-png.c b/src/cairo-png.c
-index e74a4a8..3aec86a 100644
---- a/src/cairo-png.c
-+++ b/src/cairo-png.c
-@@ -497,6 +497,20 @@ convert_bytes_to_data (png_structp png, png_row_infop row_info, png_bytep data)
- }
- }
-
-+/* branches into premultiply_data or convert_bytes_to_data depending on color type */
-+static void read_user_transform_func (png_structp png, png_row_infop row_info, png_bytep data)
-+{
-+ switch ((cairo_format_t) png_get_user_transform_ptr (png)) {
-+ case CAIRO_FORMAT_ARGB32:
-+ premultiply_data (png, row_info, data);
-+ break;
-+
-+ case CAIRO_FORMAT_RGB24:
-+ convert_bytes_to_data (png, row_info, data);
-+ break;
-+ }
-+}
-+
- static cairo_status_t
- stdio_read_func (void *closure, unsigned char *data, unsigned int size)
- {
-@@ -623,6 +637,9 @@ read_png (struct png_read_closure_t *png_closure)
-
- png_set_filler (png, 0xff, PNG_FILLER_AFTER);
-
-+ /* this must be stored before calling png_read_update_info */
-+ png_set_read_user_transform_fn (png, read_user_transform_func);
-+
- /* recheck header after setting EXPAND options */
- png_read_update_info (png, info);
- png_get_IHDR (png, info,
-@@ -643,15 +660,15 @@ read_png (struct png_read_closure_t *png_closure)
-
- case PNG_COLOR_TYPE_RGB_ALPHA:
- format = CAIRO_FORMAT_ARGB32;
-- png_set_read_user_transform_fn (png, premultiply_data);
- break;
-
- case PNG_COLOR_TYPE_RGB:
- format = CAIRO_FORMAT_RGB24;
-- png_set_read_user_transform_fn (png, convert_bytes_to_data);
- break;
- }
-
-+ png_set_user_transform_info (png, (void*) format, 0, 0);
-+
- stride = cairo_format_stride_for_width (format, png_width);
- if (stride < 0) {
- surface = _cairo_surface_create_in_error (_cairo_error (CAIRO_STATUS_INVALID_STRIDE));
---
-1.7.10.4
-
From 2dd2c826a5b367d32cf2d48ed69754795990c5db Mon Sep 17 00:00:00 2001
From: Chris Wilson <chris@chris-wilson.co.uk>
Date: Tue, 16 Apr 2013 10:58:56 +0100
--
1.7.10.4
next prev parent reply other threads:[~2013-04-17 13:54 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-17 13:34 [PATCH 1/2] png: fix over-zealous warnings about transforms Ross Burton
2013-04-17 13:34 ` Ross Burton [this message]
2013-04-17 13:46 ` [PATCH 2/2] cairo: drop transform patch, it's been rejected upstream Koen Kooi
2013-04-17 13:54 ` Burton, Ross
2013-04-17 13:48 ` [PATCH 1/2] png: fix over-zealous warnings about transforms Koen Kooi
2013-04-17 13:55 ` Burton, Ross
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1366205668-19096-2-git-send-email-ross.burton@intel.com \
--to=ross.burton@intel.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox