#1377: Cacus could/should provide M_PI --------------------------+------------------------------------------------- Reporter: sbrandt | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Cactus | Version: development version Resolution: | Keywords: GRHydro M_PI --------------------------+-------------------------------------------------
Comment (by eschnett):
Defining M_PI would lead to a failure in this code: {{{ #include <cctk_Constants.h> // defines M_PI #include <math.h> // error -- M_PI is already defined }}}
The C standard does not allow defining constants that start with "M_"; thus in this example, Cactus is at fault for violating the C standard.
Can we close this as "wontfix"?