(svn r2663) Include variables.h only in these files which need it, not globally via openttd.h

ai_new.h

@@ -2,6 +2,7 @@
 #define AI_NEW_H
 
 #include "aystar.h"
+#include "player.h"
 
 /*
  * These defines can be altered to change the behavoir of the AI

ai_old.c

@@ -13,6 +13,7 @@
 #include "economy.h"
 #include "airport.h"
 #include "depot.h"
+#include "variables.h"
 
 // remove some day perhaps?
 static Player *_cur_ai_player;

ai_pathfinder.c

@@ -6,6 +6,7 @@
 #include "command.h"
 #include "ai_new.h"
 #include "depot.h"
+#include "variables.h"
 
 #define TEST_STATION_NO_DIR 0xFF
 

airport.h

@@ -2,6 +2,7 @@
 #define AIRPORT_H
 
 #include "airport_movement.h"
+#include "variables.h"
 
 enum {MAX_TERMINALS = 6};
 enum {MAX_HELIPADS  = 2};

clear_cmd.c

@@ -5,6 +5,7 @@
 #include "tile.h"
 #include "viewport.h"
 #include "command.h"
+#include "variables.h"
 
 typedef struct TerraformerHeightMod {
 	TileIndex tile;

depot.h

@@ -6,6 +6,7 @@
 
 #include "pool.h"
 #include "tile.h"
+#include "variables.h"
 
 struct Depot {
 	TileIndex xy;

extmidi.c

@@ -6,6 +6,7 @@
 #include "hal.h"
 #include "sound.h"
 #include "string.h"
+#include "variables.h"
 #include <fcntl.h>
 #include <sys/types.h>
 #include <sys/wait.h>

fileio.c

@@ -1,6 +1,7 @@
 #include "stdafx.h"
 #include "openttd.h"
 #include "fileio.h"
+#include "variables.h"
 #if defined(UNIX) || defined(__OS2__)
 #include <ctype.h> // required for tolower()
 #endif

gfx.c

@@ -6,6 +6,7 @@
 #include "gfx.h"
 #include "table/palettes.h"
 #include "hal.h"
+#include "variables.h"
 
 Colour _cur_palette[256];
 

network.c

@@ -5,6 +5,7 @@
 #include "map.h"
 #include "network_data.h"
 #include "command.h"
+#include "variables.h"
 
 #if defined(WITH_REV)
 	extern const char _openttd_revision[];

network.h

@@ -5,6 +5,8 @@
 
 #ifdef ENABLE_NETWORK
 
+#include "player.h"
+
 // If this line is enable, every frame will have a sync test
 //  this is not needed in normal games. Normal is like 1 sync in 100
 //  frames. You can enable this if you have a lot of desyncs on a certain

network_data.c

@@ -10,6 +10,7 @@
 #include "network_client.h"
 #include "command.h"
 #include "callback_table.h"
+#include "variables.h"
 
 // This files handles the send/receive of all packets
 

network_udp.c

@@ -8,6 +8,7 @@
 #include "map.h"
 #include "network_gamelist.h"
 #include "network_udp.h"
+#include "variables.h"
 
 extern void UpdateNetworkGameWindow(bool unselect);
 extern void NetworkPopulateCompanyInfo(void);

openttd.h

@@ -72,6 +72,8 @@ typedef uint16 StringID;
 typedef uint16 SpriteID;
 typedef uint32 PalSpriteID;
 typedef uint32 CursorID;
+typedef uint16 EngineID; //! All enginenumbers should be of this type
+typedef uint16 UnitID;   //! All unitnumber stuff is of this type (or anyway, should be)
 
 typedef uint32 WindowNumber;
 typedef byte WindowClass;
@@ -546,6 +548,5 @@ enum {
 VARDEF byte _no_scroll;
 
 #include "functions.h"
-#include "variables.h"
 
 #endif /* OPENTTD_H */

pathfind.c

@@ -5,6 +5,7 @@
 #include "pathfind.h"
 #include "rail.h"
 #include "debug.h"
+#include "variables.h"
 
 // remember which tiles we have already visited so we don't visit them again.
 static bool TPFSetTileBit(TrackPathFinder *tpf, TileIndex tile, int dir)

settings.c

@@ -4,6 +4,7 @@
 #include "sound.h"
 #include "spritecache.h"
 #include "string.h"
+#include "variables.h"
 #include "table/currency.h"
 #include "network.h"
 #include "settings.h"

signs.c

@@ -5,6 +5,7 @@
 #include "saveload.h"
 #include "command.h"
 #include "strings.h"
+#include "variables.h"
 
 enum {
 	/* Max signs: 64000 (4 * 16000) */

sprite.c

@@ -4,6 +4,7 @@
 
 #include "openttd.h"
 #include "sprite.h"
+#include "variables.h"
 
 
 SpriteGroup *EvalDeterministicSpriteGroup(DeterministicSpriteGroup *dsg, int value)

texteff.c

@@ -7,6 +7,7 @@
 #include "hal.h"
 #include "console.h"
 #include "string.h"
+#include "variables.h"
 #include <stdarg.h> /* va_list */
 
 typedef struct TextEffect {

tree_cmd.c

@@ -9,6 +9,7 @@
 #include "command.h"
 #include "town.h"
 #include "sound.h"
+#include "variables.h"
 
 static int GetRandomTreeType(TileIndex tile, uint seed)
 {

variables.h

@@ -8,9 +8,6 @@
 # define MAX_PATH 260
 #endif
 
-typedef uint16 UnitID;   //! All unitnumber stuff is of this type (or anyway, should be)
-typedef uint16 EngineID; //! All enginenumbers should be of this type
-
 // Prices and also the fractional part.
 VARDEF Prices _price;
 VARDEF uint16 _price_frac[NUM_PRICES];

vehicle.h

@@ -4,6 +4,7 @@
 #include "pool.h"
 #include "order.h"
 #include "rail.h"
+#include "variables.h"
 
 enum {
 	VEH_Train = 0x10,