[ scummvm-Bugs-2016965 ] KYRA: does not compile in MSVC71

[SourceForge.net]

Bugs item #2016965, was opened at 2008-07-12 20:28
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418820&aid=2016965&group_id=37116

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Unknown Crash/Other
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Quietust (quietust)
Assigned to: Nobody/Anonymous (nobody)
Summary: KYRA: does not compile in MSVC71

Initial Comment:
"engines/kyra/sound_towns.cpp" fails to compile due to M_PI not being defined.
This appears to be due to sound_towns.cpp #defining _USE_MATH_DEFINES and then explicitly including <math.h> after all of the standard includes, even though "common/scummsys.h" (which is included by "common/system.h") already includes <math.h> on its own, causing further inclusions to have no effect.

Visual Studio 9.0 (2008) seems to have reorganized <math.h> by adding a separate include-once block around the math defines, so it compiles in that version without problems.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418820&aid=2016965&group_id=37116