diff options
| author | Jan Schmidt <thaytan@mad.scientist.com> | 2007-07-06 11:42:53 +0000 |
|---|---|---|
| committer | Jan Schmidt <thaytan@mad.scientist.com> | 2007-07-06 11:42:53 +0000 |
| commit | 5e32a710d7e28c063424d9334a74973c7ad19b0c (patch) | |
| tree | a9f7eee4627badf2b584f0d4fa2e87eb4fbe64cb /ext/libpng | |
| parent | d13ee0dff1ea31add53b5af65e9367eb68d28b51 (diff) | |
ext/libpng/gstpngdec.c: Remove endianness-flipping hack that seems to have been required only because of a bug in ffm...
Original commit message from CVS:
* 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
Diffstat (limited to 'ext/libpng')
| -rw-r--r-- | ext/libpng/gstpngdec.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/ext/libpng/gstpngdec.c b/ext/libpng/gstpngdec.c index bed0a63d..e7457817 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) || |