summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorrubidium <rubidium@openttd.org>2014-02-06 21:05:00 +0000
committerrubidium <rubidium@openttd.org>2014-02-06 21:05:00 +0000
commitc7629735a3393730d12fba678839baeec31ee72d (patch)
tree27980a0d012b67509bf1ae69aa7b974c28bcd079 /src
parentd05ff6e77b401c87b69bdc68ce2d6652f431b667 (diff)
downloadopenttd-c7629735a3393730d12fba678839baeec31ee72d.tar.xz
(svn r26312) -Codechange: iterate the nearby tiles instead of the industry list for a conflicting industry when the number of industries is really large (MJP)
Diffstat (limited to 'src')
-rw-r--r--src/industry_cmd.cpp25
1 files changed, 24 insertions, 1 deletions
diff --git a/src/industry_cmd.cpp b/src/industry_cmd.cpp
index 34096be92..bfd25fb5e 100644
--- a/src/industry_cmd.cpp
+++ b/src/industry_cmd.cpp
@@ -1563,7 +1563,30 @@ static bool CheckIfCanLevelIndustryPlatform(TileIndex tile, DoCommandFlag flags,
static CommandCost CheckIfFarEnoughFromConflictingIndustry(TileIndex tile, int type)
{
const IndustrySpec *indspec = GetIndustrySpec(type);
- const Industry *i;
+ const Industry *i = NULL;
+
+ /* On a large map with many industries, it may be faster to check an area. */
+ const int dmax = 14;
+ if (Industry::GetNumItems() > (size_t) (dmax * dmax * 2)) {
+ const int tx = TileX(tile);
+ const int ty = TileY(tile);
+ TileArea tile_area = TileArea(TileXY(max(0, tx - dmax), max(0, ty - dmax)), TileXY(min(MapMaxX(), tx + dmax), min(MapMaxY(), ty + dmax)));
+ TILE_AREA_LOOP(atile, tile_area) {
+ if (GetTileType(atile) == MP_INDUSTRY) {
+ const Industry *i2 = Industry::GetByTile(atile);
+ if (i == i2) continue;
+ i = i2;
+ if (DistanceMax(tile, i->location.tile) > dmax) continue;
+ if (i->type == indspec->conflicting[0] ||
+ i->type == indspec->conflicting[1] ||
+ i->type == indspec->conflicting[2]) {
+ return_cmd_error(STR_ERROR_INDUSTRY_TOO_CLOSE);
+ }
+ }
+ }
+ return CommandCost();
+ }
+
FOR_ALL_INDUSTRIES(i) {
/* Within 14 tiles from another industry is considered close */
if (DistanceMax(tile, i->location.tile) > 14) continue;