cpp/openttd-patchpack/source Changeset - r6106:0a01c2ed783b

Changeset - r6106:0a01c2ed783b

Parent rev.

Child rev.

[Not reviewed]

master

0 14 0

tron - 18 years ago 2007-02-22 08:43:02
tron@openttd.org

(svn r8841) -Fix

Remove {,u}intswap() and replace them by Swap()

14 files changed with 29 insertions and 34 deletions:

src/ai/default/default.cpp

src/clear_cmd.cpp

src/landscape.cpp

src/macros.h

src/newgrf_station.cpp

src/rail_cmd.cpp

src/rail_gui.cpp

src/station_cmd.cpp

src/terraform_gui.cpp

src/tree_cmd.cpp

src/tunnelbridge_cmd.cpp

src/vehicle.cpp

src/viewport.cpp

src/water_cmd.cpp

0 comments (0 inline, 0 general)

src/ai/default/default.cpp

➞

Show inline comments

@@ @@ -1546,7 +1546,7 @@ static bool AiCheckTrackResources(TileIn @@
 		w = GB(p->attr, 1, 3);
 		h = GB(p->attr, 4, 3);
-		if (p->attr & 1) uintswap(w, h);
+		if (p->attr & 1) Swap(w, h);
 		if (cargo & 0x80) {
 			GetProductionAroundTiles(values, tile2, w, h, rad);

src/clear_cmd.cpp

➞

Show inline comments

@@ @@ -370,8 +370,8 @@ int32 CmdLevelLand(TileIndex tile, uint3 @@
 	ey = TileY(tile);
 	sx = TileX(p1);
 	sy = TileY(p1);
 	if (ex < sx) intswap(ex, sx);
 	if (ey < sy) intswap(ey, sy);
 	if (ex < sx) Swap(ex, sx);
 	if (ey < sy) Swap(ey, sy);
 	tile = TileXY(sx, sy);
 	size_x = ex-sx+1;

src/landscape.cpp

➞

Show inline comments

@@ @@ -337,8 +337,8 @@ int32 CmdClearArea(TileIndex tile, uint3 @@
 	ey = TileY(tile);
 	sx = TileX(p1);
 	sy = TileY(p1);
 	if (ex < sx) intswap(ex, sx);
 	if (ey < sy) intswap(ey, sy);
 	if (ex < sx) Swap(ex, sx);
 	if (ey < sy) Swap(ey, sy);
 	money = GetAvailableMoneyForCommand();
 	cost = 0;

src/macros.h

➞

Show inline comments

@@ @@ -151,11 +151,6 @@ static inline int KillFirstBit2x64(int v @@
 #define abs myabs
 static inline int intxchg_(int *a, int b) { int t = *a; *a = b; return t; }
 #define intswap(a,b) ((b) = intxchg_(&(a), (b)))
 static inline int uintxchg_(uint *a, uint b) { uint t = *a; *a = b; return t; }
 #define uintswap(a,b) ((b) = uintxchg_(&(a), (b)))
 static inline uint16 ReadLE16Aligned(const void* x)
+{

src/newgrf_station.cpp

➞

Show inline comments

@@ @@ -200,8 +200,8 @@ uint32 GetPlatformInfo(Axis axis, byte t @@
 	uint32 retval = 0;
 	if (axis == AXIS_X) {
 		intswap(platforms, length);
 		intswap(x, y);
 		Swap(platforms, length);
 		Swap(x, y);
+	}
 	/* Limit our sizes to 4 bits */

src/rail_cmd.cpp

➞

Show inline comments

@@ @@ -914,8 +914,8 @@ int32 CmdConvertRail(TileIndex tile, uin @@
 	ey = TileY(tile);
 	sx = TileX(p1);
 	sy = TileY(p1);
 	if (ex < sx) intswap(ex, sx);
 	if (ey < sy) intswap(ey, sy);
 	if (ex < sx) Swap(ex, sx);
 	if (ey < sy) Swap(ey, sy);
 	money = GetAvailableMoneyForCommand();
 	cost = 0;

src/rail_gui.cpp

➞

Show inline comments

@@ @@ -638,11 +638,11 @@ static void HandleStationPlacement(TileI @@
 	uint ey = TileY(end);
 	uint w,h;
 	if (sx > ex) uintswap(sx,ex);
 	if (sy > ey) uintswap(sy,ey);
 	if (sx > ex) Swap(sx, ex);
 	if (sy > ey) Swap(sy, ey);
 	w = ex - sx + 1;
 	h = ey - sy + 1;
-	if (!_railstation.orientation) uintswap(w,h);
+	if (!_railstation.orientation) Swap(w, h);
 	DoCommandP(TileXY(sx, sy),
 			_railstation.orientation | (w << 8) | (h << 16),
@@ @@ -704,7 +704,7 @@ static void StationBuildWndProc(Window * @@
 		} else {
 			int x = _railstation.numtracks;
 			int y = _railstation.platlength;
-			if (_railstation.orientation == 0) intswap(x,y);
+			if (_railstation.orientation == 0) Swap(x, y);
 			if (!_remove_button_clicked)
 				SetTileSelectSize(x, y);
+		}

src/station_cmd.cpp

➞

Show inline comments

@@ @@ -2133,7 +2133,7 @@ static uint32 VehicleEnter_Station(Vehic @@
 					x &= 0xF;
 					y &= 0xF;
-					if (DiagDirToAxis(dir) != AXIS_X) intswap(x, y);
+					if (DiagDirToAxis(dir) != AXIS_X) Swap(x, y);
 					if (y == TILE_SIZE / 2) {
 						if (dir != DIAGDIR_SE && dir != DIAGDIR_SW) x = TILE_SIZE - 1 - x;
 						if (x == 12) return VETSB_ENTERED_STATION | (station_id << VETS_STATION_ID_OFFSET); /* enter station */
@@ @@ -2818,7 +2818,7 @@ static void Load_STNS(void) @@
 			uint w = GB(st->trainst_w, 4, 4);
 			uint h = GB(st->trainst_w, 0, 4);
-			if (GetRailStationAxis(st->train_tile) == AXIS_Y) uintswap(w, h);
+			if (GetRailStationAxis(st->train_tile) != AXIS_X) Swap(w, h);
 			st->trainst_w = w;
 			st->trainst_h = h;
+		}

src/terraform_gui.cpp

➞

Show inline comments

@@ @@ -40,8 +40,8 @@ static void GenerateDesertArea(TileIndex @@
 	if (_game_mode != GM_EDITOR) return;
 	if (ex < sx) intswap(ex, sx);
 	if (ey < sy) intswap(ey, sy);
 	if (ex < sx) Swap(ex, sx);
 	if (ey < sy) Swap(ey, sy);
 	size_x = (ex - sx) + 1;
 	size_y = (ey - sy) + 1;
@@ @@ -68,8 +68,8 @@ static void GenerateRockyArea(TileIndex @@
 	if (_game_mode != GM_EDITOR) return;
 	if (ex < sx) intswap(ex, sx);
 	if (ey < sy) intswap(ey, sy);
 	if (ex < sx) Swap(ex, sx);
 	if (ey < sy) Swap(ey, sy);
 	size_x = (ex - sx) + 1;
 	size_y = (ey - sy) + 1;

src/tree_cmd.cpp

➞

Show inline comments

@@ @@ -228,8 +228,8 @@ int32 CmdPlantTree(TileIndex tile, uint3 @@
 	ey = TileY(tile);
 	sx = TileX(p2);
 	sy = TileY(p2);
 	if (ex < sx) intswap(ex, sx);
 	if (ey < sy) intswap(ey, sy);
 	if (ex < sx) Swap(ex, sx);
 	if (ey < sy) Swap(ey, sy);
 	cost = 0; // total cost

src/tunnelbridge_cmd.cpp

➞

Show inline comments

@@ @@ -218,10 +218,10 @@ int32 CmdBuildBridge(TileIndex end_tile, @@
 	if (x == sx) {
 		if (y == sy) return_cmd_error(STR_5008_CANNOT_START_AND_END_ON);
 		direction = AXIS_Y;
-		if (y > sy) uintswap(y,sy);
+		if (y > sy) Swap(y, sy);
 	} else if (y == sy) {
 		direction = AXIS_X;
-		if (x > sx) uintswap(x,sx);
+		if (x > sx) Swap(x, sx);
 	} else {
 		return_cmd_error(STR_500A_START_AND_END_MUST_BE_IN);
+	}

src/vehicle.cpp

➞

Show inline comments

@@ @@ -180,8 +180,8 @@ Vehicle *FindVehicleBetween(TileIndex fr @@
 	/* Make sure x1 < x2 or y1 < y2 */
 	if (x1 > x2 || y1 > y2) {
 		intswap(x1,x2);
 		intswap(y1,y2);
 		Swap(x1, x2);
 		Swap(y1, y2);
+	}
 	FOR_ALL_VEHICLES(veh) {
 		if (without_crashed && (veh->vehstatus & VS_CRASHED) != 0) continue;

src/viewport.cpp

➞

Show inline comments

@@ @@ -1881,8 +1881,8 @@ void UpdateTileSelection(void) @@
 			x1 &= ~0xF;
 			y1 &= ~0xF;
 			if (x1 >= x2) intswap(x1,x2);
 			if (y1 >= y2) intswap(y1,y2);
 			if (x1 >= x2) Swap(x1, x2);
 			if (y1 >= y2) Swap(y1, y2);
 			_thd.new_pos.x = x1;
 			_thd.new_pos.y = y1;
 			_thd.new_size.x = x2 - x1 + TILE_SIZE;

src/water_cmd.cpp

➞

Show inline comments

@@ @@ -229,8 +229,8 @@ int32 CmdBuildCanal(TileIndex tile, uint @@
 	SET_EXPENSES_TYPE(EXPENSES_CONSTRUCTION);
 	if (x < sx) intswap(x, sx);
 	if (y < sy) intswap(y, sy);
 	if (x < sx) Swap(x, sx);
 	if (y < sy) Swap(y, sy);
 	size_x = (x - sx) + 1;
 	size_y = (y - sy) + 1;

0 comments (0 inline, 0 general)