[FFmpeg-devel] [PATCH] pgssubdec: fix subpicture output colorspace and range

[Hendrik Leppkes]

On Sat, Apr 23, 2016 at 2:33 PM, Jan Ekstrom <jeebjp at gmail.com> wrote:
> On Sat, Apr 23, 2016 at 3:21 PM, wm4 <nfxjfg at googlemail.com> wrote:
>> In that case the hdtv field should be completely removed, and the test
>> put in the palette conversion code.
>>
>
> If avctx->height is by default 0, then it would have to be something a la:
>
> if (!avctx->height || avctx->height > 576) {
>     YUV_TO_RGB1_CCIR_BT709(cb, cr);
> } else {
>     YUV_TO_RGB1_CCIR(cb, cr);
> }
>
> I guess? If the default is -1 then it'd be a check for <0 or >576.
> This way even if in a cut file a palette comes up before a
> presentation segment it should default to BT.709.
>

That looks fine.

- Hendrik