Skip to content


Browse files Browse the repository at this point in the history
Fix builds failing on BSD-like (such as OSX) due to an irrlicht bug (…
…temporary fix)

- #1687 (comment)

In case when "settings.h" is included from "emerge.cpp" or
"environment.cpp", u64 type has "unsigned long" length because
previously <stdint> was included. When "settings.h" is included from
"settings.cpp", u64 has "unsigned long long" length because no <stdint>
was included previously. This leads to different signatures of "setU64" method
and linker cannot find appropriate symbol.

The best fix of this bug should be done in the Irrlicht, but as hotfix I
think this is OK and better than types changing.

Previously this bug didn't appear because there was no "settings.cpp" file and
implementation of all methods was done in the header file.
  • Loading branch information
neoascetic authored and Zeno- committed Dec 9, 2014
1 parent 941d000 commit a1ea017
Showing 1 changed file with 11 additions and 2 deletions.
13 changes: 11 additions & 2 deletions src/irrlichttypes.h
Expand Up @@ -20,6 +20,17 @@ with this program; if not, write to the Free Software Foundation, Inc.,

/* Ensure that <stdint.h> is included before <irrTypes.h>, unless building on
* MSVC, to address an irrlicht issue:
* TODO: Decide whether or not we support non-compliant C++ compilers like old
* versions of MSCV. If we do not then <stdint.h> can always be included
* regardless of the compiler.
#ifndef _MSC_VER
# include <stdint.h>

#include <irrTypes.h>

using namespace irr;
Expand All @@ -32,11 +43,9 @@ using namespace irr;
typedef unsigned long long u64;
// Posix
#include <stdint.h>
typedef int64_t s64;
typedef uint64_t u64;


0 comments on commit a1ea017

Please sign in to comment.