drop M_PI

Author: illwieckz

src/engine/qcommon/q_shared.h

@@ -40,6 +40,7 @@ Maryland 20850 USA.
 // math.h/cmath uses _USE_MATH_DEFINES to decide if to define M_PI etc or not.
 // So define _USE_MATH_DEFINES early before including math.h/cmath
 // and before including any other header in case they bring in math.h/cmath indirectly.
+// NOTE: We don't use M_PI anymore (2024-11-25).
 #ifndef _USE_MATH_DEFINES
 #define _USE_MATH_DEFINES
 #endif
@@ -273,10 +274,6 @@ void  Com_Free_Aligned( void *ptr );
 	using fixed8_t = int;
 	using fixed16_t = int;
 
-#ifndef M_PI
-#define M_PI 3.14159265358979323846f // matches value in gcc v2 math.h
-#endif
-
 #ifndef LINE_DISTANCE_EPSILON
 #define LINE_DISTANCE_EPSILON 1e-05f
 #endif

src/engine/sys/sdl_glimp.cpp

@@ -20,7 +20,9 @@ Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 ===========================================================================
 */
 
-#include "qcommon/q_shared.h" // Include before SDL.h due to M_PI issue...
+// Include before SDL.h due to M_PI issue...
+// NOTE: We don't use M_PI anymore (2024-11-25).
+#include "qcommon/q_shared.h"
 #include <SDL.h>
 
 #ifdef USE_SMP