(svn r19814) -Codechange: give some more unnamed enums a name, in case they consisted of unrelated values use static const (u)int

src/ai/ai_instance.cpp

@@ -460,9 +460,7 @@ static const SaveLoad _ai_byte[] = {
 	SLE_END()
 };
 
-enum {
-	AISAVE_MAX_DEPTH = 25, ///< The maximum recursive depth for items stored in the savegame.
-};
+static const uint AISAVE_MAX_DEPTH = 25; ///< The maximum recursive depth for items stored in the savegame.
 
 /* static */ bool AIInstance::SaveObject(HSQUIRRELVM vm, SQInteger index, int max_depth, bool test)
 {

src/airport.h

@@ -16,14 +16,13 @@
 #include "tile_type.h"
 
 /** Some airport-related constants */
-enum {
-	MAX_TERMINALS =  10,                    ///< maximum number of terminals per airport
-	MAX_HELIPADS  =   4,                    ///< maximum number of helipads per airport
-	MAX_ELEMENTS  = 255,                    ///< maximum number of aircraft positions at airport
-	NUM_AIRPORTTILES = 256,                 ///< total number of airport tiles
-	NEW_AIRPORTTILE_OFFSET = 74,            ///< offset of first newgrf airport tile
-	INVALID_AIRPORTTILE = NUM_AIRPORTTILES, ///< id for an invalid airport tile
-};
+static const uint MAX_TERMINALS =  10;                       ///< maximum number of terminals per airport
+static const uint MAX_HELIPADS  =   4;                       ///< maximum number of helipads per airport
+static const uint MAX_ELEMENTS  = 255;                       ///< maximum number of aircraft positions at airport
+
+static const uint NUM_AIRPORTTILES       = 256;              ///< total number of airport tiles
+static const uint NEW_AIRPORTTILE_OFFSET = 74;               ///< offset of first newgrf airport tile
+static const uint INVALID_AIRPORTTILE    = NUM_AIRPORTTILES; ///< id for an invalid airport tile
 
 /** Airport types */
 enum AirportTypes {

src/airport_gui.cpp

@@ -56,7 +56,7 @@ static void PlaceAirport(TileIndex tile)
 }
 
 /** Widget number of the airport build window. */
-enum {
+enum AirportToolbarWidgets {
 	ATW_AIRPORT,
 	ATW_DEMOLISH,
 };

src/company_gui.cpp

@@ -34,12 +34,10 @@
 #include "table/strings.h"
 
 /** Company GUI constants. */
-enum {
-	FIRST_GUI_CALL = INT_MAX,  ///< default value to specify this is the first call of the resizable gui
+static const int FIRST_GUI_CALL = INT_MAX; ///< default value to specify this is the first call of the resizable gui
 
-	EXP_LINESPACE  = 2,        ///< Amount of vertical space for a horizontal (sub-)total line.
-	EXP_BLOCKSPACE = 10,       ///< Amount of vertical space between two blocks of numbers.
-};
+static const uint EXP_LINESPACE  = 2;      ///< Amount of vertical space for a horizontal (sub-)total line.
+static const uint EXP_BLOCKSPACE = 10;     ///< Amount of vertical space between two blocks of numbers.
 
 static void DoSelectCompanyManagerFace(Window *parent);
 

src/console_gui.cpp

@@ -21,13 +21,10 @@
 #include "console_func.h"
 #include "rev.h"
 
-
-enum {
-	ICON_HISTORY_SIZE       = 20,
-	ICON_LINE_SPACING       =  2,
-	ICON_RIGHT_BORDERWIDTH  = 10,
-	ICON_BOTTOM_BORDERWIDTH = 12,
-};
+static const uint ICON_HISTORY_SIZE       = 20;
+static const uint ICON_LINE_SPACING       =  2;
+static const uint ICON_RIGHT_BORDERWIDTH  = 10;
+static const uint ICON_BOTTOM_BORDERWIDTH = 12;
 
 /**
  * Container for a single line of console output
@@ -446,7 +443,7 @@ static void IConsoleHistoryNavigate(int direction)
 
 	/* watch out for overflows, just wrap around */
 	if (i < 0) i = ICON_HISTORY_SIZE - 1;
-	if (i >= ICON_HISTORY_SIZE) i = 0;
+	if ((uint)i >= ICON_HISTORY_SIZE) i = 0;
 
 	if (direction > 0) {
 		if (_iconsole_history[i] == NULL) i = 0;

src/currency.cpp

@@ -62,7 +62,7 @@ CurrencySpec _currency_specs[NUM_CURRENCY];
  * Every currency used by Ottd is there, just in case TTDPatch will
  * add those missing in its code
  **/
-enum {
+enum Currencies {
 	CURR_GBP,
 	CURR_USD,
 	CURR_EUR,

src/date.cpp

@@ -55,7 +55,7 @@ static const uint16 _month_date_from_year_day[] = {
 };
 #undef M
 
-enum {
+enum DaysTillMonth {
 	ACCUM_JAN = 0,
 	ACCUM_FEB = ACCUM_JAN + 31,
 	ACCUM_MAR = ACCUM_FEB + 29,

src/fontcache.cpp

@@ -39,10 +39,8 @@ static int _ascender[FS_END];
 
 FreeTypeSettings _freetype;
 
-enum {
-	FACE_COLOUR = 1,
-	SHADOW_COLOUR = 2,
-};
+static const byte FACE_COLOUR   = 1;
+static const byte SHADOW_COLOUR = 2;
 
 /** Get the font loaded into a Freetype face by using a font-name.
  * If no appropiate font is found, the function returns an error */

src/gamelog.cpp

@@ -85,11 +85,9 @@ void GamelogReset()
 	_current_action  = NULL;
 }
 
-enum {
-	GAMELOG_BUF_LEN = 1024 ///< length of buffer for one line of text
-};
+static const uint GAMELOG_BUF_LEN = 1024; ///< length of buffer for one line of text
 
-static int _dbgofs = 0; ///< offset in current output buffer
+static uint _dbgofs = 0; ///< offset in current output buffer
 
 static void AddDebugText(char *buf, const char *s, ...) WARN_FORMAT(2, 3);
 

src/gfx.cpp

@@ -74,10 +74,9 @@ static Rect _invalid_rect;
 static const byte *_colour_remap_ptr;
 static byte _string_colourremap[3]; ///< Recoloursprite for stringdrawing. The grf loader ensures, that ST_FONT sprites only use colours 0 to 2.
 
-enum {
-	DIRTY_BLOCK_HEIGHT   = 8,
-	DIRTY_BLOCK_WIDTH    = 64,
-};
+static const uint DIRTY_BLOCK_HEIGHT   = 8;
+static const uint DIRTY_BLOCK_WIDTH    = 64;
+
 static uint _dirty_bytes_per_line = 0;
 static byte *_dirty_blocks = NULL;
 

src/industry_cmd.cpp

@@ -2187,10 +2187,8 @@ static void ReportNewsProductionChangeIndustry(Industry *ind, CargoID type, int
 	);
 }
 
-enum {
-	PERCENT_TRANSPORTED_60 = 153,
-	PERCENT_TRANSPORTED_80 = 204,
-};
+static const uint PERCENT_TRANSPORTED_60 = 153;
+static const uint PERCENT_TRANSPORTED_80 = 204;
 
 /** Change industry production or do closure
  * @param i Industry for which changes are performed

src/misc_gui.cpp

@@ -79,14 +79,14 @@ static const WindowDesc _land_info_desc(
 );
 
 class LandInfoWindow : public Window {
-	enum {
+	enum LandInfoLines {
 		LAND_INFO_CENTERED_LINES   = 12,                       ///< Up to 12 centered lines
 		LAND_INFO_MULTICENTER_LINE = LAND_INFO_CENTERED_LINES, ///< One multicenter line
 		LAND_INFO_LINE_END,
-
-		LAND_INFO_LINE_BUFF_SIZE = 512,
 	};
 
+	static const uint LAND_INFO_LINE_BUFF_SIZE = 512;
+
 public:
 	char landinfo_data[LAND_INFO_LINE_END][LAND_INFO_LINE_BUFF_SIZE];
 	TileIndex tile;

src/music/qtmidi.cpp

@@ -41,9 +41,7 @@
 static FMusicDriver_QtMidi iFMusicDriver_QtMidi;
 
 
-enum {
-	midiType = 'Midi' ///< OSType code for MIDI songs.
-};
+static const uint MIDI_TYPE = 'Midi' ///< OSType code for MIDI songs.
 
 
 /**
@@ -61,9 +59,9 @@ static void SetMIDITypeIfNeeded(const FSRef *ref)
 	if (noErr != FSGetCatalogInfo(ref, kFSCatInfoNodeFlags | kFSCatInfoFinderInfo, &catalogInfo, NULL, NULL, NULL)) return;
 	if (!(catalogInfo.nodeFlags & kFSNodeIsDirectoryMask)) {
 		FileInfo * const info = (FileInfo *) catalogInfo.finderInfo;
-		if (info->fileType != midiType && !(info->finderFlags & kIsAlias)) {
+		if (info->fileType != MIDI_TYPE && !(info->finderFlags & kIsAlias)) {
 			OSErr e;
-			info->fileType = midiType;
+			info->fileType = MIDI_TYPE;
 			e = FSSetCatalogInfo(ref, kFSCatInfoFinderInfo, &catalogInfo);
 			if (e == noErr) {
 				DEBUG(driver, 3, "qtmidi: changed filetype to 'Midi'");
@@ -161,7 +159,7 @@ static void InitQuickTimeIfNeeded()
 
 
 /** Possible states of the QuickTime music driver. */
-enum {
+enum QTStates {
 	QT_STATE_IDLE, ///< No file loaded.
 	QT_STATE_PLAY, ///< File loaded, playing.
 	QT_STATE_STOP, ///< File loaded, stopped.

src/network/network_chat_gui.cpp

@@ -35,9 +35,7 @@
  * some spaces and possible translations of [All] to other languages. */
 assert_compile((int)DRAW_STRING_BUFFER >= (int)NETWORK_CHAT_LENGTH + NETWORK_NAME_LENGTH + 40);
 
-enum {
-	NETWORK_CHAT_LINE_SPACING = 3,
-};
+static const uint NETWORK_CHAT_LINE_SPACING = 3;
 
 struct ChatMessage {
 	char message[DRAW_STRING_BUFFER];

src/network/network_content_gui.cpp

@@ -225,10 +225,8 @@ enum NetworkContentListWindowWidgets {
 class NetworkContentListWindow : public QueryStringBaseWindow, ContentCallback {
 	typedef GUIList<const ContentInfo*> GUIContentList;
 
-	enum {
-		EDITBOX_MAX_SIZE = 50,
-		EDITBOX_MAX_LENGTH = 300,
-	};
+	static const uint EDITBOX_MAX_SIZE   =  50;
+	static const uint EDITBOX_MAX_LENGTH = 300;
 
 	/** Runtime saved values */
 	static Listing last_sorting;

src/network/network_gamelist.cpp

@@ -132,11 +132,9 @@ void NetworkGameListRemoveItem(NetworkGameList *remove)
 	}
 }
 
-enum {
-	MAX_GAME_LIST_REQUERY_COUNT  = 10, ///< How often do we requery in number of times per server?
-	REQUERY_EVERY_X_GAMELOOPS    = 60, ///< How often do we requery in time?
-	REFRESH_GAMEINFO_X_REQUERIES = 50, ///< Refresh the game info itself after REFRESH_GAMEINFO_X_REQUERIES * REQUERY_EVERY_X_GAMELOOPS game loops
-};
+static const uint MAX_GAME_LIST_REQUERY_COUNT  = 10; ///< How often do we requery in number of times per server?
+static const uint REQUERY_EVERY_X_GAMELOOPS    = 60; ///< How often do we requery in time?
+static const uint REFRESH_GAMEINFO_X_REQUERIES = 50; ///< Refresh the game info itself after REFRESH_GAMEINFO_X_REQUERIES * REQUERY_EVERY_X_GAMELOOPS game loops
 
 /** Requeries the (game) servers we have not gotten a reply from */
 void NetworkGameListRequery()

src/network/network_udp.cpp

@@ -36,11 +36,9 @@ static ThreadMutex *_network_udp_mutex = ThreadMutex::New();
 /** Session key to register ourselves to the master server */
 static uint64 _session_key = 0;
 
-enum {
-	ADVERTISE_NORMAL_INTERVAL = 30000, // interval between advertising in ticks (15 minutes)
-	ADVERTISE_RETRY_INTERVAL  =   300, // readvertise when no response after this many ticks (9 seconds)
-	ADVERTISE_RETRY_TIMES     =     3  // give up readvertising after this much failed retries
-};
+static const uint ADVERTISE_NORMAL_INTERVAL = 30000; ///< interval between advertising in ticks (15 minutes)
+static const uint ADVERTISE_RETRY_INTERVAL  =   300; ///< readvertise when no response after this many ticks (9 seconds)
+static const uint ADVERTISE_RETRY_TIMES     =     3; ///< give up readvertising after this much failed retries
 
 NetworkUDPSocketHandler *_udp_client_socket = NULL; ///< udp client socket
 NetworkUDPSocketHandler *_udp_server_socket = NULL; ///< udp server socket

src/newgrf.cpp

@@ -180,9 +180,7 @@ public:
 
 typedef void (*SpecialSpriteHandler)(ByteReader *buf);
 
-enum {
-	MAX_STATIONS = 256,
-};
+static const uint MAX_STATIONS = 256;
 
 /* Temporary data used when loading only */
 struct GRFTempEngineData {
@@ -6609,7 +6607,7 @@ static void FinaliseAirportsArray()
 
 		AirportTileSpec **&airporttilespec = (*file)->airtspec;
 		if (airporttilespec != NULL) {
-			for (int i = 0; i < NUM_AIRPORTTILES; i++) {
+			for (uint i = 0; i < NUM_AIRPORTTILES; i++) {
 				if (airporttilespec[i] != NULL && airporttilespec[i]->enabled) {
 					_airporttile_mngr.SetEntitySpec(airporttilespec[i]);
 				}

src/newgrf_engine.cpp

@@ -154,7 +154,7 @@ static int MapOldSubType(const Vehicle *v)
 
 
 /* TTDP style aircraft movement states for GRF Action 2 Var 0xE2 */
-enum {
+enum TTDPAircraftMovementStates {
 	AMS_TTDP_HANGAR,
 	AMS_TTDP_TO_HANGAR,
 	AMS_TTDP_TO_PAD1,
@@ -290,7 +290,7 @@ static byte MapAircraftMovementState(const Aircraft *v)
 
 
 /* TTDP style aircraft movement action for GRF Action 2 Var 0xE6 */
-enum {
+enum TTDPAircraftMovementActions {
 	AMA_TTDP_IN_HANGAR,
 	AMA_TTDP_ON_PAD1,
 	AMA_TTDP_ON_PAD2,

src/newgrf_gui.cpp

@@ -147,10 +147,8 @@ private:
 	const GRFConfig *sel;
 	int sel_pos;
 
-	enum {
-		EDITBOX_MAX_SIZE = 50,
-		EDITBOX_MAX_LENGTH = 300,
-	};
+	static const uint EDITBOX_MAX_SIZE   =  50;
+	static const uint EDITBOX_MAX_LENGTH = 300;
 
 	/**
 	 * (Re)build the grf as its amount has changed because

src/newgrf_station.cpp

@@ -35,9 +35,7 @@
 
 static StationClass _station_classes[STAT_CLASS_MAX];
 
-enum {
-	MAX_SPECLIST = 255,
-};
+static const uint MAX_SPECLIST = 255;
 
 enum TriggerArea {
 	TA_TILE,

src/osk_gui.cpp

@@ -51,7 +51,7 @@ enum OskWidgets {
 char _keyboard_opt[2][OSK_KEYBOARD_ENTRIES * 4 + 1];
 static WChar _keyboard[2][OSK_KEYBOARD_ENTRIES];
 
-enum {
+enum KeyStateBits {
 	KEYS_NONE,
 	KEYS_SHIFT,
 	KEYS_CAPS

src/pathfinder/npf/npf.cpp

@@ -21,15 +21,13 @@
 #include "../follow_track.hpp"
 #include "aystar.h"
 
-enum {
-	NPF_HASH_BITS = 12, ///< The size of the hash used in pathfinding. Just changing this value should be sufficient to change the hash size. Should be an even value.
-	/* Do no change below values */
-	NPF_HASH_SIZE = 1 << NPF_HASH_BITS,
-	NPF_HASH_HALFBITS = NPF_HASH_BITS / 2,
-	NPF_HASH_HALFMASK = (1 << NPF_HASH_HALFBITS) - 1
-};
+static const uint NPF_HASH_BITS = 12; ///< The size of the hash used in pathfinding. Just changing this value should be sufficient to change the hash size. Should be an even value.
+/* Do no change below values */
+static const uint NPF_HASH_SIZE = 1 << NPF_HASH_BITS;
+static const uint NPF_HASH_HALFBITS = NPF_HASH_BITS / 2;
+static const uint NPF_HASH_HALFMASK = (1 << NPF_HASH_HALFBITS) - 1;
 
-/* Meant to be stored in AyStar.targetdata */
+/** Meant to be stored in AyStar.targetdata */
 struct NPFFindStationOrTileData {
 	TileIndex dest_coords;    ///< An indication of where the station is, for heuristic purposes, or the target tile
 	StationID station_index;  ///< station index we're heading for, or INVALID_STATION when we're heading for a tile
@@ -39,21 +37,21 @@ struct NPFFindStationOrTileData {
 	const Vehicle *v;         ///< The vehicle we are pathfinding for
 };
 
-/* Indices into AyStar.userdata[] */
-enum {
+/** Indices into AyStar.userdata[] */
+enum AyStarUserDataType {
 	NPF_TYPE = 0,  ///< Contains a TransportTypes value
 	NPF_SUB_TYPE,  ///< Contains the sub transport type
 	NPF_OWNER,     ///< Contains an Owner value
 	NPF_RAILTYPES, ///< Contains a bitmask the compatible RailTypes of the engine when NPF_TYPE == TRANSPORT_RAIL. Unused otherwise.
 };
 
-/* Indices into AyStarNode.userdata[] */
-enum {
+/** Indices into AyStarNode.userdata[] */
+enum AyStarNodeUserDataType {
 	NPF_TRACKDIR_CHOICE = 0, ///< The trackdir chosen to get here
 	NPF_NODE_FLAGS,
 };
 
-/* Flags for AyStarNode.userdata[NPF_NODE_FLAGS]. Use NPFSetFlag() and NPFGetFlag() to use them. */
+/** Flags for AyStarNode.userdata[NPF_NODE_FLAGS]. Use NPFSetFlag() and NPFGetFlag() to use them. */
 enum NPFNodeFlag {
 	NPF_FLAG_SEEN_SIGNAL,       ///< Used to mark that a signal was seen on the way, for rail only
 	NPF_FLAG_2ND_SIGNAL,        ///< Used to mark that two signals were seen, rail only
@@ -66,7 +64,7 @@ enum NPFNodeFlag {
 	NPF_FLAG_IGNORE_RESERVED,   ///< Used to mark that reserved tiles should be considered impassable
 };
 
-/* Meant to be stored in AyStar.userpath */
+/** Meant to be stored in AyStar.userpath */
 struct NPFFoundTargetData {
 	uint best_bird_dist;    ///< The best heuristic found. Is 0 if the target was found
 	uint best_path_dist;    ///< The shortest path. Is UINT_MAX if no path is found

src/rail_cmd.cpp

@@ -2129,7 +2129,7 @@ static void DrawTrackBits(TileInfo *ti, TrackBits track)
  * The addtion of 2 per enum is necessary in order to "jump" over the
  * green state sprite, all signal sprites being in pair,
  * starting with the off-red state */
-enum {
+enum SignalOffsets {
 	SIGNAL_TO_SOUTHWEST =  0,
 	SIGNAL_TO_NORTHEAST =  2,
 	SIGNAL_TO_SOUTHEAST =  4,

src/signal.cpp

@@ -20,12 +20,10 @@
 
 
 /** these are the maximums used for updating signal blocks */
-enum {
-	SIG_TBU_SIZE    =  64, ///< number of signals entering to block
-	SIG_TBD_SIZE    = 256, ///< number of intersections - open nodes in current block
-	SIG_GLOB_SIZE   = 128, ///< number of open blocks (block can be opened more times until detected)
-	SIG_GLOB_UPDATE =  64, ///< how many items need to be in _globset to force update
-};
+static const uint SIG_TBU_SIZE    =  64; ///< number of signals entering to block
+static const uint SIG_TBD_SIZE    = 256; ///< number of intersections - open nodes in current block
+static const uint SIG_GLOB_SIZE   = 128; ///< number of open blocks (block can be opened more times until detected)
+static const uint SIG_GLOB_UPDATE =  64; ///< how many items need to be in _globset to force update
 
 assert_compile(SIG_GLOB_UPDATE <= SIG_GLOB_SIZE);
 

src/statusbar_gui.cpp

@@ -83,12 +83,10 @@ struct StatusBarWindow : Window {
 	int ticker_scroll;
 	int reminder_timeout;
 
-	enum {
-		TICKER_STOP    = 1640, ///< scrolling is finished when counter reaches this value
-		REMINDER_START =   91, ///< initial value of the reminder counter (right dot on the right)
-		REMINDER_STOP  =    0, ///< reminder disappears when counter reaches this value
-		COUNTER_STEP   =    2, ///< this is subtracted from active counters every tick
-	};
+	static const int TICKER_STOP    = 1640; ///< scrolling is finished when counter reaches this value
+	static const int REMINDER_START =   91; ///< initial value of the reminder counter (right dot on the right)
+	static const int REMINDER_STOP  =    0; ///< reminder disappears when counter reaches this value
+	static const int COUNTER_STEP   =    2; ///< this is subtracted from active counters every tick
 
 	StatusBarWindow(const WindowDesc *desc) : Window()
 	{

src/toolbar_gui.cpp

@@ -211,11 +211,9 @@ static void PopupMainToolbMenu(Window *w, int widget, StringID string, int count
 }
 
 /** Enum for the Company Toolbar's network related buttons */
-enum {
-	CTMN_CLIENT_LIST = -1, ///< Show the client list
-	CTMN_NEW_COMPANY = -2, ///< Create a new company
-	CTMN_SPECTATE    = -3, ///< Become spectator
-};
+static const int CTMN_CLIENT_LIST = -1; ///< Show the client list
+static const int CTMN_NEW_COMPANY = -2; ///< Create a new company
+static const int CTMN_SPECTATE    = -3; ///< Become spectator
 
 /**
  * Pop up a generic company list menu.

src/train_gui.cpp

@@ -126,10 +126,8 @@ struct CargoSummaryItem {
 	}
 };
 
-enum {
-	TRAIN_DETAILS_MIN_INDENT = 32, ///< Minimum indent level in the train details window
-	TRAIN_DETAILS_MAX_INDENT = 72, ///< Maximum indent level in the train details window; wider than this and we start on a new line
-};
+static const uint TRAIN_DETAILS_MIN_INDENT = 32; ///< Minimum indent level in the train details window
+static const uint TRAIN_DETAILS_MAX_INDENT = 72; ///< Maximum indent level in the train details window; wider than this and we start on a new line
 
 /** Container for the cargo summary information. */
 typedef SmallVector<CargoSummaryItem, 2> CargoSummary;
@@ -307,7 +305,7 @@ void DrawTrainDetails(const Train *v, int left, int right, int y, int vscroll_po
 			GetCargoSummaryOfArticulatedVehicle(v, &_cargo_summary);
 
 			/* Draw sprites */
-			int dx = 0;
+			uint dx = 0;
 			int px = x;
 			const Train *u = v;
 			do {

src/video/allegro_v.cpp

@@ -318,10 +318,8 @@ static uint32 ConvertAllegroKeyIntoMy()
 	return (key << 16) + unicode;
 }
 
-enum {
-	LEFT_BUTTON,
-	RIGHT_BUTTON,
-};
+static const uint LEFT_BUTTON  = 0;
+static const uint RIGHT_BUTTON = 1;
 
 static void PollEvent()
 {

src/video/cocoa/event.mm

@@ -49,7 +49,7 @@
 
 
 /* Right Mouse Button Emulation enum */
-enum {
+enum RightMouseButtonEmulationState {
 	RMBE_COMMAND,
 	RMBE_CONTROL,
 	RMBE_OFF,