cpp/openttd-patchpack/source Changeset - r11231:db031b23683b

Changeset - r11231:db031b23683b

Parent rev.

Child rev.

[Not reviewed]

master

0 4 0

yexo - 15 years ago 2009-02-25 21:50:54
yexo@openttd.org

(svn r15585) -Codechange: Remove TILE_ASSERT and replace all instances with assert(tile < MapSize()).

4 files changed with 7 insertions and 12 deletions:

src/industry_cmd.cpp

src/map_func.h

src/town_cmd.cpp

src/unmovable_cmd.cpp

0 comments (0 inline, 0 general)

src/industry_cmd.cpp

➞

Show inline comments

@@ @@ -928,7 +928,7 @@ static void PlantFarmField(TileIndex til @@
 	/* check the amount of bad tiles */
 	count = 0;
 	BEGIN_TILE_LOOP(cur_tile, size_x, size_y, tile)
-		TILE_ASSERT(cur_tile);
+		assert(cur_tile < MapSize());
 		count += IsBadFarmFieldTile(cur_tile);
 	END_TILE_LOOP(cur_tile, size_x, size_y, tile)
 	if (count * 2 >= size_x * size_y) return;
@@ @@ -940,7 +940,7 @@ static void PlantFarmField(TileIndex til @@
 	/* make field */
 	BEGIN_TILE_LOOP(cur_tile, size_x, size_y, tile)
-		TILE_ASSERT(cur_tile);
+		assert(cur_tile < MapSize());
 		if (!IsBadFarmFieldTile2(cur_tile)) {
 			MakeField(cur_tile, field_type, industry);
 			SetClearCounter(cur_tile, counter);

src/map_func.h

➞

Show inline comments

@@ @@ -18,11 +18,6 @@ extern uint _map_tile_mask; @@
  */
 #define TILE_MASK(x) ((x) & _map_tile_mask)
 /**
  * Asserts when the tile is outside of the map.
  * @param x the tile to check
  */
 #define TILE_ASSERT(x) assert(TILE_MASK(x) == (x));
 /**
  * Pointer to the tile-array.

src/town_cmd.cpp

➞

Show inline comments

@@ @@ -818,7 +818,7 @@ no_slope: @@
 static bool TerraformTownTile(TileIndex tile, int edges, int dir)
+{
-	TILE_ASSERT(tile);
+	assert(tile < MapSize());
 	CommandCost r = DoCommand(tile, edges, dir, DC_AUTO | DC_NO_WATER, CMD_TERRAFORM_LAND);
 	if (CmdFailed(r) || r.GetCost() >= (_price.terraform + 2) * 8) return false;
@@ @@ -828,7 +828,7 @@ static bool TerraformTownTile(TileIndex @@
 static void LevelTownLand(TileIndex tile)
+{
-	TILE_ASSERT(tile);
+	assert(tile < MapSize());
 	/* Don't terraform if land is plain or if there's a house there. */
 	if (IsTileType(tile, MP_HOUSE)) return;
@@ @@ -1027,7 +1027,7 @@ static void GrowTownInTile(TileIndex *ti @@
 	RoadBits rcmd = ROAD_NONE;  // RoadBits for the road construction command
 	TileIndex tile = *tile_ptr; // The main tile on which we base our growth
-	TILE_ASSERT(tile);
+	assert(tile < MapSize());
 	if (cur_rb == ROAD_NONE) {
 		/* Tile has no road. First reset the status counter
@@ @@ -1195,7 +1195,7 @@ static int GrowTownAtRoad(Town *t, TileI @@
 	 */
 	DiagDirection target_dir = DIAGDIR_END; // The direction in which we want to extend the town
-	TILE_ASSERT(tile);
+	assert(tile < MapSize());
 	/* Number of times to search.
 	 * Better roads, 2X2 and 3X3 grid grow quite fast so we give

src/unmovable_cmd.cpp

➞

Show inline comments

@@ @@ -452,7 +452,7 @@ void GenerateUnmovables() @@
 				MakeLighthouse(tile);
 				IncreaseGeneratingWorldProgress(GWP_UNMOVABLE);
 				lighthouses_to_build--;
-				TILE_ASSERT(tile);
+				assert(tile < MapSize());
 				break;
+			}
 			tile = AddTileIndexDiffCWrap(tile, TileIndexDiffCByDiagDir(dir));

0 comments (0 inline, 0 general)