[FFmpeg-devel] [PATCH] cos_tablegen: fix the lack on M_PI

Carl Eugen Hoyos cehoyos at ag.or.at
Sat Jan 5 14:28:30 CET 2013


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

> > -        double freq = 2*M_PI/m;
> > +        double freq = 2*3.14159265358979323846/m;
> >          printf("%s(%i) = {\n   ", do_sin ? "SINTABLE" : "COSTABLE", m);
> >          for (j = 0; j < m/2 - 1; j++) {
> >              int idx = j > m/4 ? m/2 - j : j;
> 
> #include <math.h> // M_PI
> 
>  ... looks better to me.

It appears that there are configurations where this 
is not sufficient.
(-D__ANSI_something)

Carl Eugen



More information about the ffmpeg-devel mailing list