earthdistance update for msvc

Started by Magnus Haganderover 19 years ago3 messageshackers
Jump to latest
#1Magnus Hagander
magnus@hagander.net

A define is needed to expose the M_PI define from the system headers.

//Magnu <<earthdistance.diff>> s

Attachments:

earthdistance.diffapplication/octet-stream; name=earthdistance.diffDownload+3-0
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Magnus Hagander (#1)
Re: earthdistance update for msvc

"Magnus Hagander" <mha@sollentuna.net> 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

#3Magnus Hagander
magnus@hagander.net
In reply to: Tom Lane (#2)
Re: earthdistance update for msvc

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