[FFmpeg-cvslog] Fix linking without yasm.

Carl Eugen Hoyos cehoyos at ag.or.at
Sat Mar 24 23:45:52 CET 2012


Clément Bœsch <ubitux <at> gmail.com> writes:

> > -dsp->put_no_rnd_vc1_chroma_pixels_tab[0]= ff_put_vc1_chroma_mc8_mmx_nornd;
> > +if (HAVE_YASM)
> > + dsp->put_no_rnd_vc1_chroma_pixels_tab[0]= ff_put_vc1_chroma_mc8_mmx_nornd;
> 
> I may miss something obvious but isn't this abusing the fact the compiler
> will optimize out this branch? 

We are using this fact on many places in the code, 
it is seen as a feature, 

> Aren't we supposed to use the cpp here?

because it allows to check for syntax etc. because we do not 
(only) use cpp.

(I have no strong opinion on this, but above is what was 
written many times here afair.)

Carl Eugen



More information about the ffmpeg-cvslog mailing list