[FFmpeg-cvslog] pgssub: Fix subpicture colorspace and range

Jan Ekström git at videolan.org
Tue Jun 21 22:22:19 CEST 2016


ffmpeg | branch: master | Jan Ekström <jeebjp at gmail.com> | Sun Apr 24 20:33:45 2016 +0200| [885a9d6087315a85d98f7e89656ef01dc7104c4c] | committer: Luca Barbato

pgssub: Fix subpicture colorspace and range

Widen the values from limited to full range and use BT.709 where it
should be used according to the video resolution:

SD is BT.601, HD is BT.709

Default to BT.709 due to most observed HDMV content being HD.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=885a9d6087315a85d98f7e89656ef01dc7104c4c
---

 libavcodec/pgssubdec.c |   10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/libavcodec/pgssubdec.c b/libavcodec/pgssubdec.c
index 7bcb1ed..a4cb66b 100644
--- a/libavcodec/pgssubdec.c
+++ b/libavcodec/pgssubdec.c
@@ -351,8 +351,14 @@ static int parse_palette_segment(AVCodecContext *avctx,
         cb        = bytestream_get_byte(&buf);
         alpha     = bytestream_get_byte(&buf);
 
-        YUV_TO_RGB1(cb, cr);
-        YUV_TO_RGB2(r, g, b, y);
+        /* Default to BT.709 colorspace. In case of <= 576 height use BT.601 */
+        if (avctx->height <= 0 || avctx->height > 576) {
+            YUV_TO_RGB1_CCIR_BT709(cb, cr);
+        } else {
+            YUV_TO_RGB1_CCIR(cb, cr);
+        }
+
+        YUV_TO_RGB2_CCIR(r, g, b, y);
 
         ff_dlog(avctx, "Color %d := (%d,%d,%d,%d)\n", color_id, r, g, b, alpha);
 



More information about the ffmpeg-cvslog mailing list