summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog7
-rw-r--r--ext/libpng/gstpngdec.c7
2 files changed, 13 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index e27bd8f47..c4f6d8c89 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2007-07-06 Jan Schmidt <thaytan@mad.scientist.com>
+
+ * ext/libpng/gstpngdec.c: (gst_pngdec_caps_create_and_set):
+ Remove endianness-flipping hack that seems to have been required
+ only because of a bug in ffmpegcolorspace.
+ Partially Fixes: #451908
+
2007-07-05 Stefan Kost <ensonic@users.sf.net>
* docs/plugins/Makefile.am:
diff --git a/ext/libpng/gstpngdec.c b/ext/libpng/gstpngdec.c
index bed0a63d6..e74578171 100644
--- a/ext/libpng/gstpngdec.c
+++ b/ext/libpng/gstpngdec.c
@@ -357,9 +357,14 @@ gst_pngdec_caps_create_and_set (GstPngDec * pngdec)
/* Get Color type */
color_type = png_get_color_type (pngdec->png, pngdec->info);
- /* HACK: The doc states that it's RGBA but apparently it's not... */
+#if 0
+ /* We used to have this HACK to reverse the outgoing bytes, but the problem
+ * that originally required the hack seems to have been in ffmpegcolorspace's
+ * RGBA descriptions. It doesn't seem needed now that's fixed, but might
+ * still be needed on big-endian systems, I'm not sure. J.S. 6/7/2007 */
if (color_type == PNG_COLOR_TYPE_RGB_ALPHA)
png_set_bgr (pngdec->png);
+#endif
/* Gray scale converted to RGB and upscaled to 8 bits */
if ((color_type == PNG_COLOR_TYPE_GRAY_ALPHA) ||