On Sat, Apr 23, 2016 at 2:33 PM, Jan Ekstrom <jee...@gmail.com> wrote:
> On Sat, Apr 23, 2016 at 3:21 PM, wm4 <nfx...@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
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
