widelands-dev team mailing list archive

[Tino <tino.miegel@xxxxxxxxxxxxxx>]

Tino has proposed merging lp:~widelands-dev/widelands/missing_include_win32 into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/missing_include_win32/+merge/186245

Ok, the latest code does only compile with this additional include on win32.
Could someone do a quick check if this does break anything for linux/macos? Or do we have to add #ifdef win32?
Or is this also the problem that the latest trunk cannot be build by the launchpad build system?
-- 
https://code.launchpad.net/~widelands-dev/widelands/missing_include_win32/+merge/186245
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/missing_include_win32 into lp:widelands.

=== modified file 'src/logic/map_revision.h'
--- src/logic/map_revision.h	2013-09-14 19:10:06 +0000
+++ src/logic/map_revision.h	2013-09-18 06:21:32 +0000
@@ -21,6 +21,7 @@
 #define MAP_REVISION_H
 
 #include <cstring>
+#include <ctime>
 #include <memory>
 #include <set>
 #include <string>