summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authoryexo <yexo@openttd.org>2009-02-14 21:16:21 +0000
committeryexo <yexo@openttd.org>2009-02-14 21:16:21 +0000
commit16954ba37806a4510e626d5ceb29f8114c275a3d (patch)
tree0080f4623a8ee03c92d581adb1d7f6de837db053
parentbce1bcb0f27affa6678c3f657c06955afbf4b47b (diff)
downloadopenttd-16954ba37806a4510e626d5ceb29f8114c275a3d.tar.xz
(svn r15489) -Change [API CHANGE]: Remove AIAirport::AirportAvailable. AIAirport::IsValidAirportType now only returns true for available AirportTypes.
-rw-r--r--bin/ai/regression/regression.nut1
-rw-r--r--bin/ai/regression/regression.txt75
-rw-r--r--src/ai/api/ai_airport.cpp9
-rw-r--r--src/ai/api/ai_airport.hpp10
-rw-r--r--src/ai/api/ai_airport.hpp.sq1
5 files changed, 35 insertions, 61 deletions
diff --git a/bin/ai/regression/regression.nut b/bin/ai/regression/regression.nut
index 679aa0bec..eaa7c88a7 100644
--- a/bin/ai/regression/regression.nut
+++ b/bin/ai/regression/regression.nut
@@ -214,7 +214,6 @@ function Regression::Airport()
for (local i = -1; i < 10; i++) {
print(" IsValidAirportType(" + i + "): " + AIAirport.IsValidAirportType(i));
- print(" AirportAvailable(" + i + "): " + AIAirport.AirportAvailable(i));
print(" GetAirportWidth(" + i + "): " + AIAirport.GetAirportWidth(i));
print(" GetAirportHeight(" + i + "): " + AIAirport.GetAirportHeight(i));
print(" GetAirportCoverageRadius(" + i + "): " + AIAirport.GetAirportCoverageRadius(i));
diff --git a/bin/ai/regression/regression.txt b/bin/ai/regression/regression.txt
index 83328f659..d23e5451e 100644
--- a/bin/ai/regression/regression.txt
+++ b/bin/ai/regression/regression.txt
@@ -576,57 +576,46 @@
GetHangarOfAirport(): -1
GetAirportType(): 255
IsValidAirportType(-1): false
- AirportAvailable(-1): false
GetAirportWidth(-1): -1
GetAirportHeight(-1): -1
GetAirportCoverageRadius(-1): -1
IsValidAirportType(0): true
- AirportAvailable(0): true
GetAirportWidth(0): 4
GetAirportHeight(0): 3
GetAirportCoverageRadius(0): 4
- IsValidAirportType(1): true
- AirportAvailable(1): false
- GetAirportWidth(1): 6
- GetAirportHeight(1): 6
- GetAirportCoverageRadius(1): 5
- IsValidAirportType(2): true
- AirportAvailable(2): false
- GetAirportWidth(2): 1
- GetAirportHeight(2): 1
- GetAirportCoverageRadius(2): 4
- IsValidAirportType(3): true
- AirportAvailable(3): false
- GetAirportWidth(3): 6
- GetAirportHeight(3): 6
- GetAirportCoverageRadius(3): 6
- IsValidAirportType(4): true
- AirportAvailable(4): false
- GetAirportWidth(4): 7
- GetAirportHeight(4): 7
- GetAirportCoverageRadius(4): 8
- IsValidAirportType(5): true
- AirportAvailable(5): false
- GetAirportWidth(5): 5
- GetAirportHeight(5): 4
- GetAirportCoverageRadius(5): 4
- IsValidAirportType(6): true
- AirportAvailable(6): false
- GetAirportWidth(6): 2
- GetAirportHeight(6): 2
- GetAirportCoverageRadius(6): 4
- IsValidAirportType(7): true
- AirportAvailable(7): false
- GetAirportWidth(7): 9
- GetAirportHeight(7): 11
- GetAirportCoverageRadius(7): 10
- IsValidAirportType(8): true
- AirportAvailable(8): false
- GetAirportWidth(8): 4
- GetAirportHeight(8): 2
- GetAirportCoverageRadius(8): 4
+ IsValidAirportType(1): false
+ GetAirportWidth(1): -1
+ GetAirportHeight(1): -1
+ GetAirportCoverageRadius(1): -1
+ IsValidAirportType(2): false
+ GetAirportWidth(2): -1
+ GetAirportHeight(2): -1
+ GetAirportCoverageRadius(2): -1
+ IsValidAirportType(3): false
+ GetAirportWidth(3): -1
+ GetAirportHeight(3): -1
+ GetAirportCoverageRadius(3): -1
+ IsValidAirportType(4): false
+ GetAirportWidth(4): -1
+ GetAirportHeight(4): -1
+ GetAirportCoverageRadius(4): -1
+ IsValidAirportType(5): false
+ GetAirportWidth(5): -1
+ GetAirportHeight(5): -1
+ GetAirportCoverageRadius(5): -1
+ IsValidAirportType(6): false
+ GetAirportWidth(6): -1
+ GetAirportHeight(6): -1
+ GetAirportCoverageRadius(6): -1
+ IsValidAirportType(7): false
+ GetAirportWidth(7): -1
+ GetAirportHeight(7): -1
+ GetAirportCoverageRadius(7): -1
+ IsValidAirportType(8): false
+ GetAirportWidth(8): -1
+ GetAirportHeight(8): -1
+ GetAirportCoverageRadius(8): -1
IsValidAirportType(9): false
- AirportAvailable(9): false
GetAirportWidth(9): -1
GetAirportHeight(9): -1
GetAirportCoverageRadius(9): -1
diff --git a/src/ai/api/ai_airport.cpp b/src/ai/api/ai_airport.cpp
index b2b969b82..ed58701c3 100644
--- a/src/ai/api/ai_airport.cpp
+++ b/src/ai/api/ai_airport.cpp
@@ -11,7 +11,7 @@
/* static */ bool AIAirport::IsValidAirportType(AirportType type)
{
- return type >= AT_SMALL && type <= AT_HELISTATION;
+ return type >= AT_SMALL && type <= AT_HELISTATION && HasBit(::GetValidAirports(), type);
}
/* static */ bool AIAirport::IsHangarTile(TileIndex tile)
@@ -28,13 +28,6 @@
return ::IsTileType(tile, MP_STATION) && ::IsAirport(tile);
}
-/* static */ bool AIAirport::AirportAvailable(AirportType type)
-{
- if (!IsValidAirportType(type)) return false;
-
- return HasBit(::GetValidAirports(), type);
-}
-
/* static */ int32 AIAirport::GetAirportWidth(AirportType type)
{
if (!IsValidAirportType(type)) return -1;
diff --git a/src/ai/api/ai_airport.hpp b/src/ai/api/ai_airport.hpp
index 559d9c6ae..88d208043 100644
--- a/src/ai/api/ai_airport.hpp
+++ b/src/ai/api/ai_airport.hpp
@@ -47,9 +47,9 @@ public:
};
/**
- * Checks whether the given AirportType is valid.
+ * Checks whether the given AirportType is valid and available.
* @param type The AirportType to check.
- * @return True if and only if the AirportTypeis valid.
+ * @return True if and only if the AirportType is valid and available.
*/
static bool IsValidAirportType(AirportType type);
@@ -70,12 +70,6 @@ public:
static bool IsAirportTile(TileIndex tile);
/**
- * Check if a certain airport type is already available.
- * @param type The type of airport to check.
- */
- static bool AirportAvailable(AirportType type);
-
- /**
* Get the width of this type of airport.
* @param type The type of airport.
* @return The width in tiles.
diff --git a/src/ai/api/ai_airport.hpp.sq b/src/ai/api/ai_airport.hpp.sq
index f84412869..fdf8153b6 100644
--- a/src/ai/api/ai_airport.hpp.sq
+++ b/src/ai/api/ai_airport.hpp.sq
@@ -41,7 +41,6 @@ void SQAIAirport_Register(Squirrel *engine) {
SQAIAirport.DefSQStaticMethod(engine, &AIAirport::IsValidAirportType, "IsValidAirportType", 2, "?i");
SQAIAirport.DefSQStaticMethod(engine, &AIAirport::IsHangarTile, "IsHangarTile", 2, "?i");
SQAIAirport.DefSQStaticMethod(engine, &AIAirport::IsAirportTile, "IsAirportTile", 2, "?i");
- SQAIAirport.DefSQStaticMethod(engine, &AIAirport::AirportAvailable, "AirportAvailable", 2, "?i");
SQAIAirport.DefSQStaticMethod(engine, &AIAirport::GetAirportWidth, "GetAirportWidth", 2, "?i");
SQAIAirport.DefSQStaticMethod(engine, &AIAirport::GetAirportHeight, "GetAirportHeight", 2, "?i");
SQAIAirport.DefSQStaticMethod(engine, &AIAirport::GetAirportCoverageRadius, "GetAirportCoverageRadius", 2, "?i");