cpp/openttd-patchpack/source Changeset - r5602:8fa1fb36cb68

Changeset - r5602:8fa1fb36cb68

Parent rev.

Child rev.

[Not reviewed]

master

0 5 0

celestar - 18 years ago 2007-01-11 12:32:10
celestar@openttd.org

(svn r8056) -Codechange: Remove swap_byte, swap_byte, and siblings (some were not used anyway) and replace them by our SwapT implementation

5 files changed with 15 insertions and 22 deletions:

src/ai/default/default.cpp

src/macros.h

src/oldloader.cpp

src/train_cmd.cpp

src/viewport.cpp

0 comments (0 inline, 0 general)

src/ai/default/default.cpp

➞

Show inline comments

@@ @@ -2291,10 +2291,10 @@ static void AiStateBuildRail(Player *p) @@
 		AiBuildRail(p);
 		// Alternate between edges
 		swap_tile(&p->ai.start_tile_a, &p->ai.start_tile_b);
 		swap_tile(&p->ai.cur_tile_a, &p->ai.cur_tile_b);
 		swap_byte(&p->ai.start_dir_a, &p->ai.start_dir_b);
 		swap_byte(&p->ai.cur_dir_a, &p->ai.cur_dir_b);
 		SwapT(&p->ai.start_tile_a, &p->ai.start_tile_b);
 		SwapT(&p->ai.cur_tile_a, &p->ai.cur_tile_b);
 		SwapT(&p->ai.start_dir_a, &p->ai.start_dir_b);
 		SwapT(&p->ai.cur_dir_a, &p->ai.cur_dir_b);
 		return;
+	}
@@ @@ -3112,10 +3112,10 @@ static void AiStateBuildRoad(Player *p) @@
 		AiBuildRoad(p);
 		// Alternate between edges
 		swap_tile(&p->ai.start_tile_a, &p->ai.start_tile_b);
 		swap_tile(&p->ai.cur_tile_a, &p->ai.cur_tile_b);
 		swap_byte(&p->ai.start_dir_a, &p->ai.start_dir_b);
 		swap_byte(&p->ai.cur_dir_a, &p->ai.cur_dir_b);
 		SwapT(&p->ai.start_tile_a, &p->ai.start_tile_b);
 		SwapT(&p->ai.cur_tile_a, &p->ai.cur_tile_b);
 		SwapT(&p->ai.start_dir_a, &p->ai.start_dir_b);
 		SwapT(&p->ai.cur_dir_a, &p->ai.cur_dir_b);
 		return;
+	}

src/macros.h

➞

Show inline comments

@@ @@ -140,13 +140,6 @@ static inline int intxchg_(int *a, int 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 void swap_byte(byte *a, byte *b) { byte t = *a; *a = *b; *b = t; }
 static inline void swap_uint16(uint16 *a, uint16 *b) { uint16 t = *a; *a = *b; *b = t; }
 static inline void swap_int16(int16 *a, int16 *b) { int16 t = *a; *a = *b; *b = t; }
 static inline void swap_uint32(uint32 *a, uint32 *b) { uint32 t = *a; *a = *b; *b = t; }
 static inline void swap_int32(int32 *a, int32 *b) { int32 t = *a; *a = *b; *b = t; }
 static inline void swap_tile(TileIndex *a, TileIndex *b) { TileIndex t = *a; *a = *b; *b = t; }
 static inline uint16 ReadLE16Aligned(const void* x)
+{

src/oldloader.cpp

➞

Show inline comments

@@ @@ -310,7 +310,7 @@ static void FixOldStations(void) @@
 	FOR_ALL_STATIONS(st) {
 		/* Check if we need to swap width and height for the station */
 		if (st->train_tile != 0 && GetRailStationAxis(st->train_tile) != AXIS_X) {
-			swap_byte(&st->trainst_w, &st->trainst_h);
+			SwapT(&st->trainst_w, &st->trainst_h);
+		}
 		/* Check if there is a bus or truck station, and convert to new format */

src/train_cmd.cpp

➞

Show inline comments

@@ @@ -1603,10 +1603,10 @@ static void ReverseTrainSwapVeh(Vehicle @@
 		if (!(b->u.rail.track & 0x80)) b->direction = ReverseDir(b->direction);
 		/* swap more variables */
 		swap_int32(&a->x_pos, &b->x_pos);
 		swap_int32(&a->y_pos, &b->y_pos);
 		swap_tile(&a->tile, &b->tile);
 		swap_byte(&a->z_pos, &b->z_pos);
 		SwapT(&a->x_pos, &b->x_pos);
 		SwapT(&a->y_pos, &b->y_pos);
 		SwapT(&a->tile, &b->tile);
 		SwapT(&a->z_pos, &b->z_pos);
 		SwapTrainFlags(&a->u.rail.flags, &b->u.rail.flags);

src/viewport.cpp

➞

Show inline comments

@@ @@ -2043,7 +2043,7 @@ static int CalcHeightdiff(HighLightStyle @@
 	uint h0, h1, ht; // start heigth, end height, and temp variable
 	if (start_tile == end_tile) return 0;
-	if (swap) swap_tile(&start_tile, &end_tile);
+	if (swap) SwapT(&start_tile, &end_tile);
 	switch (style & HT_DRAG_MASK) {
 		case HT_RECT: {
@@ @@ -2103,7 +2103,7 @@ static int CalcHeightdiff(HighLightStyle @@
 		} break;
+	}
-	if (swap) swap_uint32(&h0, &h1);
+	if (swap) SwapT(&h0, &h1);
 	/* Minimap shows height in intervals of 50 meters, let's do the same */
 	return (int)(h1 - h0) * 50;
+}

0 comments (0 inline, 0 general)