@@ -18,11 +18,11 @@ with this program; if not, write to the Free Software Foundation, Inc.,
18
18
*/
19
19
20
20
#ifdef _MSC_VER
21
- #ifndef SERVER // Dedicated server isn't linked with Irrlicht
22
- #pragma comment(lib, "Irrlicht.lib")
23
- // This would get rid of the console window
24
- // #pragma comment(linker, "/subsystem:windows /ENTRY:mainCRTStartup")
25
- #endif
21
+ #ifndef SERVER // Dedicated server isn't linked with Irrlicht
22
+ #pragma comment(lib, "Irrlicht.lib")
23
+ // This would get rid of the console window
24
+ // #pragma comment(linker, "/subsystem:windows /ENTRY:mainCRTStartup")
25
+ #endif
26
26
#pragma comment(lib, "zlibwapi.lib")
27
27
#pragma comment(lib, "Shell32.lib")
28
28
#endif
@@ -59,7 +59,14 @@ with this program; if not, write to the Free Software Foundation, Inc.,
59
59
#endif
60
60
61
61
#ifdef HAVE_TOUCHSCREENGUI
62
- #include " touchscreengui.h"
62
+ #include " touchscreengui.h"
63
+ #endif
64
+
65
+ #if !defined(SERVER) && \
66
+ (IRRLICHT_VERSION_MAJOR == 1 ) && \
67
+ (IRRLICHT_VERSION_MINOR == 8 ) && \
68
+ (IRRLICHT_VERSION_REVISION == 2 )
69
+ #error "Irrlicht 1.8.2 is known to be broken - please update Irrlicht to version >= 1.8.3"
63
70
#endif
64
71
65
72
#define DEBUGFILE " debug.txt"
4 commit comments
sofar commentedon Nov 9, 2015
+1 ;^)
HybridDog commentedon Nov 10, 2015
please test the other way round (
IRRLICHT_VERSION_REVISION == 2
first, …), l guess 1.8.* often appears but * .*.2 seldom appears.est31 commentedon Nov 13, 2015
I'd consider that less readable. The way @kwolekr did it was completely okay.
est31 commentedon Nov 13, 2015
And don't overcomplicate version recognition #ifdefs, after all a broken ifdef version recognition is why 1.8.2 is a broken release.