[MEncoder-users] Mencoder soft telecine broken; fix included

xiphmont at xiph.org xiphmont at xiph.org
Fri Sep 28 23:50:02 CEST 2007


On 9/28/07, Nico Sabbi <nicola_sabbi at fastwebnet.it> wrote:
> Il Friday 28 September 2007 07:01:10 xiphmont at xiph.org ha scritto:

> committed, but with a fix:
>
> -  if(((s->buffer[i+4] & 0xf0) == 0x20))
> + if(((s->buffer[i+4] & 0xf0) == 0x10))
>    {
>             se_ptr = &(s->buffer[i+4]);
>             if(priv->patch_sde)
>                 patch_panscan(priv, se_ptr);
>
>
> this is wrong: se_ptr must point to the sequence_extension (0x10),
> while patch_panscan must work on the sequence_display_extension.

Ah!  And now I see how the telecine bug crept in.  Thank you for the review.

Monty



More information about the MEncoder-users mailing list