Re: [FFmpeg-devel] [PATCH] vf_colorspace: Add support for smpte248 color primaries

2017-06-07 Thread Vittorio Giovara
On Wed, Jun 7, 2017 at 7:36 PM, Kieran O Leary wrote: > Hi, > > Should the vf colorspace documentation be updated with this new addition? Oh right, I forgot sorry. Do you want to do the honors since I forgot also about gbr? -- Vittorio ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] vf_colorspace: Add support for smpte248 color primaries

2017-06-07 Thread Kieran O Leary
Hi, Should the vf colorspace documentation be updated with this new addition? Best, Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] vf_colorspace: Add support for smpte248 color primaries

2017-06-07 Thread Ronald S. Bultje
Hi, On Wed, Jun 7, 2017 at 5:50 PM, Vittorio Giovara wrote: > +ENUM("smpte248", AVCOL_PRI_SMPTE428, "prm"), 428 or 248? Ronald ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] vf_colorspace: Add support for smpte248 color primaries

2017-06-07 Thread Vittorio Giovara
--- I don't have a sample for this, but I used the values found in mpv (and I find the source very reliable about this). Vittorio libavfilter/vf_colorspace.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c index 0b1bc81f99..91fa44