[Developers] Re: [Aeidevelopment] CVS: "AEIDevelopment/ADMMass/src volume_integral.c"

Jonathan Thornburg jthorn at aei.mpg.de
Thu Oct 6 04:27:38 CDT 2005


Hi, Erik,

On Wed, 5 Oct 2005, Erik Schnetter wrote:
> Update of /numrelcvs/AEIDevelopment/ADMMass/src
> In directory cvsserv:/tmp/cvs-serv30475/src
>
> Modified Files:
> 	volume_integral.c
> Log Message:
> #include <math.h> for sqrt()
> #define PI to be more accurate (but then, who cares...)

> #include <math.h>
> #include <stdio.h>
> #include <stdlib.h>
> 
> #include <cctk.h>
> #include <cctk_Arguments.h>
> #include <cctk_Parameters.h>
> 
> #include "SpaceMask.h"
> 
> #define PI 3.141592653589793238462643383279502884197169399375105820974944592307816406286208998628034825342117068

Why not just use M_PI (defined by <math.h>, which you've already included)?
Are there systems whose <math.h> doesn't define M_PI?

ciao,

-- 
-- Jonathan Thornburg <jthorn at aei.mpg.de>
    Max-Planck-Institut fuer Gravitationsphysik (Albert-Einstein-Institut),
    Golm, Germany, "Old Europe"     http://www.aei.mpg.de/~jthorn/home.html
    "Washing one's hands of the conflict between the powerful and the
     powerless means to side with the powerful, not to be neutral."
                                       -- quote by Freire / poster by Oxfam




More information about the Developers mailing list