Re: [FFmpeg-devel] [PATCH] [dnxhddec] Do not overwrite colorspace if the container has set it.

2017-11-28 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 06:43:32PM -0800, Steven Robertson wrote: > It's... less wrong. > > The VC-3 DNxHR update added a 'clv' field to an existing byte in the frame > header to allow for the carriage of a limited set of color info (0 = > 709/709, 1=2020/2020NCL, 2=2020/2020CL, 3=container

Re: [FFmpeg-devel] [PATCH] [dnxhddec] Do not overwrite colorspace if the container has set it.

2017-11-27 Thread Steven Robertson
It's... less wrong. The VC-3 DNxHR update added a 'clv' field to an existing byte in the frame header to allow for the carriage of a limited set of color info (0 = 709/709, 1=2020/2020NCL, 2=2020/2020CL, 3=container specifies), but because that field is new-ish and assigns a meaning other than

Re: [FFmpeg-devel] [PATCH] [dnxhddec] Do not overwrite colorspace if the container has set it.

2017-11-27 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 12:36:48AM -0800, Steven Robertson wrote: > Signed-off-by: Steven Robertson > --- > libavcodec/dnxhddec.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c > index

[FFmpeg-devel] [PATCH] [dnxhddec] Do not overwrite colorspace if the container has set it.

2017-11-27 Thread Steven Robertson
Signed-off-by: Steven Robertson --- libavcodec/dnxhddec.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c index f46e41a456..6f8c716412 100644 --- a/libavcodec/dnxhddec.c +++ b/libavcodec/dnxhddec.c @@ -93,7