Re: [PATCHES] earthdistance update for msvc

2006-10-19 Thread Tom Lane
Magnus Hagander [EMAIL PROTECTED] writes:
 A define is needed to expose the M_PI define from the system headers.

It seems like the other places where we depend on M_PI, we instead have

#ifndef M_PI
#define M_PI 3.14159265358979323846
#endif

Perhaps that's a better solution?

regards, tom lane

---(end of broadcast)---
TIP 1: if posting/reading through Usenet, please send an appropriate
   subscribe-nomail command to [EMAIL PROTECTED] so that your
   message can get through to the mailing list cleanly


Re: [PATCHES] earthdistance update for msvc

2006-10-19 Thread Magnus Hagander
  A define is needed to expose the M_PI define from the 
 system headers.
 
 It seems like the other places where we depend on M_PI, we 
 instead have
 
 #ifndef M_PI
 #define M_PI 3.14159265358979323846
 #endif
 
 Perhaps that's a better solution?

Oh. I actually did that for my quick-n-dirty fix and thought it would be
rejected so I researched the other one :-) either one is fine by me.

//Magnus

---(end of broadcast)---
TIP 2: Don't 'kill -9' the postmaster