[FFmpeg-devel] [PATCH]lavc/pgssubdec: Fix palette colourspace

Hendrik Leppkes h.leppkes at gmail.com
Sun Apr 17 23:42:39 CEST 2016


On Sun, Apr 17, 2016 at 9:16 PM, Jan Ekstrom <jeebjp at gmail.com> wrote:
> On Sun, Apr 17, 2016 at 10:08 PM, Carl Eugen Hoyos <cehoyos at 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


More information about the ffmpeg-devel mailing list