cpp/openttd-patchpack/source Changeset - r14168:135d7d8d5e5b

Changeset - r14168:135d7d8d5e5b

Parent rev.

Child rev.

[Not reviewed]

master

0 8 0

rubidium - 15 years ago 2010-01-04 21:10:20
rubidium@openttd.org

(svn r18726) -Fix [FS#3463]: with non-uniform industries the 'supplies' text when building a station could be incorrect (missing a cargo)
-Change [NoAI]: AITile::GetCargoProduction now returns the number of producers and not the number of tiles of producers.
-Fix [NoAI]: AITileList_IndustryProducing would omit some tiles for at which a station would get cargo.

8 files changed with 151 insertions and 122 deletions:

bin/ai/regression/regression.txt

src/ai/api/ai_changelog.hpp

src/ai/api/ai_tile.hpp

src/ai/api/ai_tilelist.cpp

src/industry_cmd.cpp

src/station.cpp

src/station_cmd.cpp

src/tilearea_type.h

0 comments (0 inline, 0 general)

bin/ai/regression/regression.txt

➞

Show inline comments

@@ @@ -7818,103 +7818,105 @@ @@
 => 8
 => 8
 => 8
 => 8
 --TileList_IndustryProducing--
-  Count():             90
+  Count():             92
   Location ListDump:
 => 14
 => 14
 => 14
 => 14
 => 14
 => 14
 => 14
 => 14
 => 12
 => 12
 => 12
 => 12
 => 12
 => 12
 => 11
 => 11
 => 10
 => 10
 => 10
 => 10
 => 9
 => 9
 => 9
 => 9
 => 9
 => 9
 => 9
 => 9
 => 8
 => 8
 => 8
 => 8
 => 8
 => 8
 => 8
 => 8
 => 7
 => 7
 => 6
 => 6
 => 6
 => 6
 => 6
 => 6
 => 6
 => 6
 => 5
 => 5
 => 5
 => 5
 => 4
 => 4
 => 4
 => 4
 => 4
 => 4
 => 4
 => 4
 => 4
 => 4
 => 4
 => 4
 => 4
 => 4
 => 3
 => 3
 => 3
 => 3
 => 3
 => 3
 => 3
 => 3
 => 2
 => 2
 => 2
 => 2
 => 2
 => 2
 => 2
 => 2
 => 2
 => 2
 => 2
 => 2
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 => 1
 --TileList_StationType--
   Count():             4
   Location ListDump:
 => 0

src/ai/api/ai_changelog.hpp

➞

Show inline comments

@@ @@ -88,12 +88,18 @@ @@
  * \li AIIndustryType::GetConstructionCost() now returns -1 if the industry is
  *     neither buildable nor prospectable.
  * \li AIEngine::IsValidEngine will now return true if you have at least one
  *     vehicle of that type in your company, regardless if it's still buildable
  *     or not. AIEngine::IsBuildable returns only true when you can actually
  *     build an engine.
  * \li AITile::GetCargoProduction will now return the number of producers,
  *     including houses instead the number of producing tiles. This means that
  *     also industries that do not have a tile within the radius, but where
  *     the search bounding box and the industry's bounding box intersect, are
  *     counted. Previously these industries (and their cargos), although they
  *     produced cargo for a station at the given location, were not returned.
+ *
  * \b 0.7.5
+ *
  * No changes
+ *
  * \b 0.7.4

src/ai/api/ai_tile.hpp

➞

Show inline comments

@@ @@ -306,26 +306,24 @@ public: @@
 	 * @pre radius >= 0.
 	 * @return Value below 8 means no acceptance; the more the better.
 	 */
 	static int32 GetCargoAcceptance(TileIndex tile, CargoID cargo_type, int width, int height, int radius);
 	/**
 	 * Checks how many tiles in the radius produces this cargo.
 	 *  It creates a radius around the tile, and adds up all tiles that produce
 	 *  this cargo.
 	 * Checks how many producers in the radius produces this cargo.
 	 *  It creates a radius around the tile, and counts all producer of this cargo.
 	 * @param tile The tile to check on.
 	 * @param cargo_type The cargo to check the production of.
 	 * @param width The width of the station.
 	 * @param height The height of the station.
 	 * @param radius The radius of the station.
 	 * @pre AIMap::IsValidTile(tile).
 	 * @pre width > 0.
 	 * @pre height > 0.
 	 * @pre radius >= 0.
 	 * @return The tiles that produce this cargo within radius of the tile.
 	 * @note Town(houses) are not included in the value.
 	 * @return The number of producers that produce this cargo within radius of the tile.
 	 */
 	static int32 GetCargoProduction(TileIndex tile, CargoID cargo_type, int width, int height, int radius);
 	/**
 	 * Get the manhattan distance from the tile to the tile.
 	 * @param tile_from The tile to get the distance to.

src/ai/api/ai_tilelist.cpp

➞

Show inline comments

@@ @@ -87,38 +87,25 @@ AITileList_IndustryProducing::AITileList @@
+{
 	if (!AIIndustry::IsValidIndustry(industry_id) || radius <= 0) return;
 	const Industry *i = ::Industry::Get(industry_id);
 	/* Check if this industry produces anything */
+	{
 		bool cargo_produces = false;
 		for (byte j = 0; j < lengthof(i->produced_cargo); j++) {
 			if (i->produced_cargo[j] != CT_INVALID) cargo_produces = true;
+		}
 		if (!cargo_produces) return;
 	bool cargo_produces = false;
 	for (byte j = 0; j < lengthof(i->produced_cargo); j++) {
 		if (i->produced_cargo[j] != CT_INVALID) cargo_produces = true;
+	}
 	if (!cargo_produces) return;
 	if (!_settings_game.station.modified_catchment) radius = CA_UNMODIFIED;
 	TILE_LOOP(cur_tile, i->location.w + radius * 2, i->location.h + radius * 2, i->location.tile - ::TileDiffXY(radius, radius)) {
 		if (!::IsValidTile(cur_tile)) continue;
 		/* Exclude all tiles that belong to this industry */
 		if (::IsTileType(cur_tile, MP_INDUSTRY) && ::GetIndustryIndex(cur_tile) == industry_id) continue;
 		/* Only add the tile if it produces the cargo (a bug in OpenTTD makes this
 		 *  inconsitance). */
 		CargoArray produced = ::GetProductionAroundTiles(cur_tile, 1, 1, radius);
+		{
 			bool cargo_produces = false;
 			for (byte j = 0; j < lengthof(i->produced_cargo); j++) {
 				if (i->produced_cargo[j] != CT_INVALID && produced[i->produced_cargo[j]] != 0) cargo_produces = true;
+			}
 			if (!cargo_produces) continue;
+		}
 		this->AddTile(cur_tile);
+	}
+}
 AITileList_StationType::AITileList_StationType(StationID station_id, AIStation::StationType station_type)
+{

src/industry_cmd.cpp

➞

Show inline comments

@@ @@ -921,22 +921,12 @@ static bool ClickTile_Industry(TileIndex @@
 static TrackStatus GetTileTrackStatus_Industry(TileIndex tile, TransportType mode, uint sub_mode, DiagDirection side)
+{
 	return 0;
+}
 static void AddProducedCargo_Industry(TileIndex tile, CargoArray &produced)
+{
 	const Industry *i = Industry::GetByTile(tile);
 	for (uint j = 0; j < lengthof(i->produced_cargo); j++) {
 		CargoID cargo = i->produced_cargo[j];
 		if (cargo != CT_INVALID) produced[cargo]++;
+	}
+}
 static void ChangeTileOwner_Industry(TileIndex tile, Owner old_owner, Owner new_owner)
+{
 	/* If the founder merges, the industry was created by the merged company */
 	Industry *i = Industry::GetByTile(tile);
 	if (i->founder == old_owner) i->founder = (new_owner == INVALID_OWNER) ? OWNER_NONE : new_owner;
+}
@@ @@ -2465,11 +2455,11 @@ extern const TileTypeProcs _tile_type_in @@
 	GetTileDesc_Industry,        // get_tile_desc_proc
 	GetTileTrackStatus_Industry, // get_tile_track_status_proc
 	ClickTile_Industry,          // click_tile_proc
 	AnimateTile_Industry,        // animate_tile_proc
 	TileLoop_Industry,           // tile_loop_proc
 	ChangeTileOwner_Industry,    // change_tile_owner_proc
-	AddProducedCargo_Industry,   // add_produced_cargo_proc
+	NULL,                        // add_produced_cargo_proc
 	NULL,                        // vehicle_enter_tile_proc
 	GetFoundation_Industry,      // get_foundation_proc
 	TerraformTile_Industry,      // terraform_tile_proc
 };

src/station.cpp

➞

Show inline comments

@@ @@ -536,11 +536,35 @@ void TileArea::Add(TileIndex to_add) @@
 	this->tile = TileXY(sx, sy);
 	this->w    = ex - sx + 1;
 	this->h    = ey - sy + 1;
+}
 bool TileArea::Intersects(const TileArea &ta) const
+{
 	if (ta.w == 0 || this->w == 0) return false;
 	assert(ta.w != 0 && ta.h != 0 && this->w != 0 && this->h != 0);
 	uint left1   = TileX(this->tile);
 	uint top1    = TileY(this->tile);
 	uint right1  = left1 + this->w - 1;
 	uint bottom1 = top1  + this->h - 1;
 	uint left2   = TileX(ta.tile);
 	uint top2    = TileY(ta.tile);
 	uint right2  = left2 + ta.w - 1;
 	uint bottom2 = top2  + ta.h - 1;
 	return !(
 			left2   > right1  ||
 			right2  < left1   ||
 			top2    > bottom1 ||
 			bottom2 < top1
 		);
+}
 void InitializeStations()
+{
 	_station_pool.CleanPool();
+}

src/station_cmd.cpp

➞

Show inline comments

@@ @@ -482,16 +482,31 @@ CargoArray GetProductionAroundTiles(Tile @@
 	assert(x1 < x2);
 	assert(y1 < y2);
 	assert(w > 0);
 	assert(h > 0);
 	for (int yc = y1; yc != y2; yc++) {
 		for (int xc = x1; xc != x2; xc++) {
 			TileIndex tile = TileXY(xc, yc);
 			AddProducedCargo(tile, produced);
 	TileArea ta(TileXY(x1, y1), TileXY(x2 - 1, y2 - 1));
 	/* Loop over all tiles to get the produced cargo of
 	 * everything except industries */
 	TILE_AREA_LOOP(tile, ta) AddProducedCargo(tile, produced);
 	/* Loop over the industries. They produce cargo for
 	 * anything that is within 'rad' from their bounding
 	 * box. As such if you have e.g. a oil well the tile
 	 * area loop might not hit an industry tile while
 	 * the industry would produce cargo for the station.
 	 */
 	const Industry *i;
 	FOR_ALL_INDUSTRIES(i) {
 		if (!ta.Intersects(i->location)) continue;
 		for (uint j = 0; j < lengthof(i->produced_cargo); j++) {
 			CargoID cargo = i->produced_cargo[j];
 			if (cargo != CT_INVALID) produced[cargo]++;
+		}
+	}
 	return produced;
+}

src/tilearea_type.h

➞

Show inline comments

@@ @@ -50,9 +50,16 @@ struct TileArea { @@
 	void Clear()
+	{
 		this->tile = INVALID_TILE;
 		this->w    = 0;
 		this->h    = 0;
+	}
 	/**
 	 * Does this tile area intersect with another?
 	 * @param ta the other tile area to check against.
 	 * @return true if they intersect.
 	 */
 	bool Intersects(const TileArea &ta) const;
 };
 #endif /* TILEAREA_TYPE_H */

0 comments (0 inline, 0 general)