On Sun, Apr 17, 2016 at 9:16 PM, Jan Ekstrom <jee...@gmail.com> wrote: > On Sun, Apr 17, 2016 at 10:08 PM, Carl Eugen Hoyos <ceho...@ag.or.at> wrote: >> >> Does attached make it better? > > So the difference between the two conversion functions is that one is > Rec.601 in limited range, and the other is Rec.709 in limited range? > If so, that seems correct. The actual coded width/height of "625/50" > can only be 720x576, so you can modify the check for that height :) .
No, both functions are Rec.601, CCIR produces full-range RGB and the other one limited range. Its important to note here that "limited range RGB" is really not something that is typically considered valid, the range is a YUV concept and converting to RGB should generally use the entire RGB range. So in short, the original patch "improves" it to some degree, as the PAL8 format that subtitles come in should be considered a full-range RGB format, but its still not the correct HD RGB colorspace. - Hendrik _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
