[FFmpeg-devel] [PATCH] h264: don't touch H264Context->ref_count[] during MB decoding.

Ronald S. Bultje rsbultje at gmail.com
Thu Oct 4 16:51:18 CEST 2012


Hi,

On Oct 4, 2012 7:16 AM, "Clément Bœsch" <ubitux at gmail.com> wrote:
>
> On Thu, Oct 04, 2012 at 04:13:24PM +0200, Clément Bœsch wrote:
> [...]
> > > >      if(MB_MBAFF){
> > > > -        h->ref_count[0] <<= 1;
> > > > -        h->ref_count[1] <<= 1;
> > > > +        local_ref_count[0] <<= 1;
> > > > +        local_ref_count[1] <<= 1;
> > > >      }
> > >
> > > maybe
> > > local_ref_count[0] = h->ref_count[0] << MB_MBAFF
> > > is faster but otherwise whatever is fast & clean is fine with me
> > >
> >
> > Ah right, I'll push the attached version tomorrow unless there is an
> > objection then.
> >
>
> Derp, sent a bad draft version. Fixed.

Has anyone been able to test if this decreases the amount of fate failures?

Ronald


More information about the ffmpeg-devel mailing list