Remove some redundant #ifdefs and added some new ones

Sources/CMakeLists.txt

@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 2.8.7)
+cmake_minimum_required(VERSION 3.7.2)
 project(SeriousEngine)
 
 # Set @rpath for Mac OS X shared library install names.

Sources/Ecc/CMakeLists.txt

@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 2.8.7)
+cmake_minimum_required(VERSION 3.7.2)
 project(Ecc)
 
 if(MSVC)

Sources/Engine/Base/ErrorReporting.cpp

@@ -26,10 +26,6 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 
 #include <Engine/Graphics/Adapter.h>
 
-#ifndef PLATFORM_WIN32
-#include "SDL.h"
-#endif
-
 INDEX con_bNoWarnings = 0;
 
 // global handle for application window in full-screen mode

Sources/Engine/Base/Win32/Win32DynamicLoader.cpp

@@ -2,6 +2,8 @@
 
 /* rcg10072001 Implemented. */
 
+#ifdef PLATFORM_WIN32
+
 #include <Engine/Engine.h>
 #include <Engine/Base/DynamicLoader.h>
 
@@ -83,6 +85,7 @@ CWin32DynamicLoader::~CWin32DynamicLoader(void)
         ::FreeLibrary(module);
 }
 
+#endif //PLATFORM_WIN32
 
 // end of Win32DynamicLoader.cpp ...
 

Sources/Engine/Base/Win32/Win32FileSystem.cpp

@@ -4,6 +4,8 @@
 
 // !!! FIXME: rcg10142001 This should really be using CTStrings...
 
+#ifdef PLATFORM_WIN32
+
 #include <io.h>
 
 #include <Engine/Engine.h>
@@ -116,6 +118,6 @@ CDynamicArray<CTString> CWin32FileSystem::FindFiles(const char *dir,
     return(retval);
 }
 
+#endif //PLATFORM_WIN32
+
 // end of Win32FileSystem.cpp ...
-
-

Sources/Engine/Base/Win32/Win32Input.cpp

@@ -2,6 +2,8 @@
 
 /* rcg10072001 Moved stuff into this file. */
 
+#ifdef PLATFORM_WIN32
+
 #include <Engine/Base/Timer.h>
 #include <Engine/Base/Input.h>
 #include <Engine/Base/Translation.h>
@@ -982,5 +984,7 @@ LONG CInput::PlatformGetJoystickCount(void)
   return((LONG) joyGetNumDevs());
 }
 
+#endif //PLATFORM_WIN32
+
 // end of Win32Input.cpp ...
 

Sources/Engine/Base/Win32/Win32Synchronization.cpp

@@ -13,6 +13,8 @@ You should have received a copy of the GNU General Public License along
 with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. */
 
+#ifdef PLATFORM_WIN32
+
 #include "Engine/StdH.h"
 
 #include <Engine/Base/Synchronization.h>
@@ -317,3 +319,4 @@ void CTSingleLock::Unlock(void)
   sl_bLocked = FALSE;
 }
 
+#endif //PLATFORM_WIN32

Sources/Engine/Base/Win32/Win32Timer.cpp

@@ -2,6 +2,8 @@
 
 /* rcg10072001 Moved stuff into this file. */
 
+#ifdef PLATFORM_WIN32
+
 #include <Engine/Engine.h>
 #include <Engine/Base/Timer.h>
 
@@ -10,6 +12,8 @@ void CTimer::Sleep(DWORD milliseconds)
     ::Sleep(milliseconds);
 }
 
+#endif //PLATFORM_WIN32
+
 // end of Win32Timer.cpp ...
 
 

Sources/Engine/Engine.cpp

@@ -45,9 +45,6 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include <Engine/Templates/StaticArray.cpp>
 #include <Engine/Base/IFeel.h>
 
-#if PLATFORM_UNIX
-#include "SDL.h"
-#endif
 
 // this version string can be referenced from outside the engine
 ENGINE_API CTString _strEngineBuild  = "";

Sources/Engine/Graphics/Adapter.cpp

@@ -22,9 +22,9 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 
 
 // !!! FIXME : rcg11052001 move this somewhere.
-#ifdef PLATFORM_UNIX
-#include "SDL.h"
-#endif
+//#ifdef PLATFORM_UNIX
+//#include "SDL.h"
+//#endif
 
 extern BOOL _bDedicatedServer;
 #ifdef SE1_D3D

Sources/Engine/Graphics/GfxLibrary.cpp

@@ -52,11 +52,6 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 // !!! FIXME: rcg10112001  driver type is valid.
 
 
-// !!! FIXME: rcg11052001  ... and I could get rid of this, too...
-#ifdef PLATFORM_UNIX
-#include <SDL.h>
-#endif
-
 // control for partial usage of compiled vertex arrays
 BOOL CVA_b2D     = FALSE;
 BOOL CVA_bWorld  = FALSE;

Sources/Engine/Graphics/Gfx_Direct3D.cpp

@@ -15,6 +15,8 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 
 #include "Engine/StdH.h"
 
+#ifdef SE1_D3D
+
 #include <Engine/Base/Translation.h>
 #include <Engine/Base/ErrorReporting.h>
 #include <Engine/Base/Memory.h>
@@ -27,7 +29,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include <Engine/Templates/DynamicContainer.cpp>
 #include <Engine/Templates/Stock_CTextureData.h>
 
-#ifdef SE1_D3D
+
 
 
 // asm shortcuts

Sources/Engine/Graphics/Win32/Win32Adapter.cpp

@@ -1,5 +1,7 @@
 /* Copyright (c) 2002-2012 Croteam Ltd. All rights reserved. */
 
+#ifdef PLATFORM_WIN32
+
 #include <windows.h>
 
 ULONG DetermineDesktopWidth(void)
@@ -7,4 +9,4 @@ ULONG DetermineDesktopWidth(void)
   return((ULONG) ::GetSystemMetrics(SM_CXSCREEN));
 }
 
-
+#endif

Sources/Engine/Graphics/Win32/Win32OpenGL.cpp

@@ -1,5 +1,7 @@
 /* Copyright (c) 2002-2012 Croteam Ltd. All rights reserved. */
 
+#ifdef PLATFORM_WIN32
+
 #include <Engine/Engine.h>
 
 static void FailFunction_t(const char *strName) {
@@ -438,3 +440,4 @@ BOOL CGfxLibrary::SetCurrentViewport_OGL(CViewPort *pvp)
   return TRUE;
 }
 
+#endif // PLATFORM_WIN32

Sources/Engine/Sound/SoundLibrary.cpp

@@ -25,10 +25,6 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "initguid.h"
 #endif
 
-// !!! FIXME : Move all the SDL stuff to a different file...
-#ifdef PLATFORM_UNIX
-#include "SDL.h"
-#endif
 
 #include <Engine/Engine.h>
 #include <Engine/Sound/SoundLibrary.h>

Sources/Engine/World/WorldIO.cpp

@@ -197,7 +197,7 @@ void CWorld::ReadBrushes_t( CTStream *istrm)// throw char *
   wo_baBrushes.Read_t(istrm);
   CallProgressHook_t(1.0f);
 
-  // if there are some terrais in world
+  // if there are some terrains in world
   if(istrm->PeekID_t()==CChunkID("TRAR")) { // 'terrain archive'
     SetProgressDescription(TRANS("loading terrains"));
     CallProgressHook_t(0.0f);