Hi,

On Tue, Apr 4, 2017 at 6:33 AM, Carl Eugen Hoyos <ceho...@ag.or.at> wrote:

> Hi!
>
> I believe attached patch makes FFmpeg a little more consistent with itself.


Looks like it, yes.

This patch does expose a bug we've discussed before though. Since the
return value of handle_jpeg() is or()'ed into src_range and dst_range, it
overwrites an explicit zero range requested by the user. So If my input is
a h264 grayscale file with color_range in VUI explicitly set to MPEG-range,
I believe swscale does the wrong thing and the output color will be
distorted.

(This is orthogonal to your patch, but may be worth of a trac ticket.)

Ronald
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to