cpp/openttd-patchpack/source Changeset - r9298:06e4a9703da1

Changeset - r9298:06e4a9703da1

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

rubidium - 16 years ago 2008-05-18 17:40:13
rubidium@openttd.org

(svn r13166) -Codechange: CmdSetRoadDriveSide belongs in road_cmd.cpp, not settings_gui.cpp.

2 files changed with 32 insertions and 31 deletions:

src/road_cmd.cpp

src/settings_gui.cpp

0 comments (0 inline, 0 general)

src/road_cmd.cpp

➞

Show inline comments

@@ @@ -43,6 +43,37 @@ @@
 #include "table/sprites.h"
 #include "table/strings.h"
 bool RoadVehiclesAreBuilt()
+{
 	const Vehicle* v;
 	FOR_ALL_VEHICLES(v) {
 		if (v->type == VEH_ROAD) return true;
+	}
 	return false;
+}
 /**
  * Change the side of the road vehicles drive on (server only).
  * @param tile unused
  * @param flags operation to perform
  * @param p1 the side of the road; 0 = left side and 1 = right side
  * @param p2 unused
  */
 CommandCost CmdSetRoadDriveSide(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
+{
 	/* Check boundaries and you can only change this if NO vehicles have been built yet,
 	 * except in the intro-menu where of course it's always possible to do so. */
 	if (p1 > 1 || (_game_mode != GM_MENU && RoadVehiclesAreBuilt())) return CMD_ERROR;
 	if (flags & DC_EXEC) {
 		_opt_ptr->road_side = p1;
 		InvalidateWindow(WC_GAME_OPTIONS, 0);
+	}
 	return CommandCost();
+}
 #define M(x) (1 << (x))
 /* Level crossings may only be built on these slopes */
 static const uint32 VALID_LEVEL_CROSSING_SLOPES = (M(SLOPE_SEN) | M(SLOPE_ENW) | M(SLOPE_NWS) | M(SLOPE_NS) | M(SLOPE_WSE) | M(SLOPE_EW) | M(SLOPE_FLAT));

src/settings_gui.cpp

➞

Show inline comments

@@ @@ -20,7 +20,6 @@ @@
 #include "strings_func.h"
 #include "functions.h"
 #include "window_func.h"
 #include "vehicle_base.h"
 #include "core/alloc_func.hpp"
 #include "string_func.h"
 #include "gfx_func.h"
@@ @@ -102,17 +101,6 @@ static int GetCurRes() @@
 	return i;
+}
 static inline bool RoadVehiclesAreBuilt()
+{
 	const Vehicle* v;
 	FOR_ALL_VEHICLES(v) {
 		if (v->type == VEH_ROAD) return true;
+	}
 	return false;
+}
 enum GameOptionsWidgets {
 	GAMEOPT_CURRENCY_BTN    =  4,
 	GAMEOPT_DISTANCE_BTN    =  6,
@@ @@ -199,6 +187,7 @@ struct GameOptionsWindow : Window { @@
 			case GAMEOPT_ROADSIDE_BTN: { // Setup road-side dropdown
 				int i = 0;
 				extern bool RoadVehiclesAreBuilt();
 				/* You can only change the drive side if you are in the menu or ingame with
 				 * no vehicles present. In a networking game only the server can change it */
@@ @@ -323,25 +312,6 @@ struct GameOptionsWindow : Window { @@
+	}
 };
 /** Change the side of the road vehicles drive on (server only).
  * @param tile unused
  * @param flags operation to perform
  * @param p1 the side of the road; 0 = left side and 1 = right side
  * @param p2 unused
  */
 CommandCost CmdSetRoadDriveSide(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
+{
 	/* Check boundaries and you can only change this if NO vehicles have been built yet,
 	 * except in the intro-menu where of course it's always possible to do so. */
 	if (p1 > 1 || (_game_mode != GM_MENU && RoadVehiclesAreBuilt())) return CMD_ERROR;
 	if (flags & DC_EXEC) {
 		_opt_ptr->road_side = p1;
 		InvalidateWindow(WC_GAME_OPTIONS, 0);
+	}
 	return CommandCost();
+}
 static const Widget _game_options_widgets[] = {
 {   WWT_CLOSEBOX,   RESIZE_NONE,    14,     0,    10,     0,    13, STR_00C5,                          STR_018B_CLOSE_WINDOW},
 {    WWT_CAPTION,   RESIZE_NONE,    14,    11,   369,     0,    13, STR_00B1_GAME_OPTIONS,             STR_018C_WINDOW_TITLE_DRAG_THIS},

0 comments (0 inline, 0 general)