[FFmpeg-devel] PATCH libswscale yuv2rgbtst infastructure

Ivo ivop
Tue May 1 18:55:28 CEST 2007


On Tuesday 01 May 2007 18:35, Michael Niedermayer wrote:
> On Tue, May 01, 2007 at 09:36:24AM -0400, Marc Hoffman wrote:
> > I'm also thinking that we should check this
> >
> >     sws->flags |=  ACCL;
> >     csc_accl =  yuv2rgb_get_func_ptr (sws);
> >     sws->flags &=  ~(ACCL);
> >     csc_c    =  yuv2rgb_get_func_ptr (sws);
> >
> > +    if (csc_c == csc_accl) {
> > +       av_log (0,0, "%s: no accelerator found\n", sws_format_name
> > (c->dstFormat)) +       return 0;
> > +    }
> >
> >     csc_accl(sws, src, strides, 0, h   , dst, dstrides);
> >     csc_c   (sws, src, strides, 0, h   , ref, dstrides);
> >
> > What do you think?
>
> good idea

I have something similar in my local version of cs_test to verify that the C 
and SIMD output are identical. That's how I found the last few bugs that 
were fixed. I'm not sure if it's suitable for submission though. You think 
I should clean it up so it can be a permanent check of cs_test?

--Ivo




More information about the ffmpeg-devel mailing list