summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorplanetmaker <planetmaker@openttd.org>2014-05-06 20:53:08 +0000
committerplanetmaker <planetmaker@openttd.org>2014-05-06 20:53:08 +0000
commit68af7a843cc910865c1e19b8ec672dd88f9bfb24 (patch)
treefad022ecd59ae268761b18274d882d385849fda4 /src
parent34b1d89dbe9534c1215c2d599cbf23fae2d068c4 (diff)
downloadopenttd-68af7a843cc910865c1e19b8ec672dd88f9bfb24.tar.xz
(svn r26567) -Cleanup: Remove unused StringID offset in orders check
Diffstat (limited to 'src')
-rw-r--r--src/lang/english.txt4
-rw-r--r--src/order_cmd.cpp22
2 files changed, 10 insertions, 16 deletions
diff --git a/src/lang/english.txt b/src/lang/english.txt
index 81ada1d9f..15de85ec2 100644
--- a/src/lang/english.txt
+++ b/src/lang/english.txt
@@ -810,14 +810,12 @@ STR_NEWS_ROAD_VEHICLE_IS_WAITING :{WHITE}{VEHICLE
STR_NEWS_SHIP_IS_WAITING :{WHITE}{VEHICLE} is waiting in depot
STR_NEWS_AIRCRAFT_IS_WAITING :{WHITE}{VEHICLE} is waiting in the aircraft hangar
-# Start of order review system
-# DON'T ADD OR REMOVE LINES HERE
+# Order review system / warnings
STR_NEWS_VEHICLE_HAS_TOO_FEW_ORDERS :{WHITE}{VEHICLE} has too few orders in the schedule
STR_NEWS_VEHICLE_HAS_VOID_ORDER :{WHITE}{VEHICLE} has a void order
STR_NEWS_VEHICLE_HAS_DUPLICATE_ENTRY :{WHITE}{VEHICLE} has duplicate orders
STR_NEWS_VEHICLE_HAS_INVALID_ENTRY :{WHITE}{VEHICLE} has an invalid station in its orders
STR_NEWS_PLANE_USES_TOO_SHORT_RUNWAY :{WHITE}{VEHICLE} has an airport with a too short runway in its orders
-# end of order system
STR_NEWS_VEHICLE_IS_GETTING_OLD :{WHITE}{VEHICLE} is getting old
STR_NEWS_VEHICLE_IS_GETTING_VERY_OLD :{WHITE}{VEHICLE} is getting very old
diff --git a/src/order_cmd.cpp b/src/order_cmd.cpp
index 2fcc7bdcd..f3a6d91d6 100644
--- a/src/order_cmd.cpp
+++ b/src/order_cmd.cpp
@@ -1772,17 +1772,16 @@ void CheckOrders(const Vehicle *v)
/* Only check every 20 days, so that we don't flood the message log */
if (v->owner == _local_company && v->day_counter % 20 == 0) {
- int n_st, problem_type = -1;
const Order *order;
- int message = 0;
+ StringID message = INVALID_STRING_ID;
/* Check the order list */
- n_st = 0;
+ int n_st = 0;
FOR_VEHICLE_ORDERS(v, order) {
/* Dummy order? */
if (order->IsType(OT_DUMMY)) {
- problem_type = 1;
+ message = STR_NEWS_VEHICLE_HAS_VOID_ORDER;
break;
}
/* Does station have a load-bay for this vehicle? */
@@ -1791,14 +1790,14 @@ void CheckOrders(const Vehicle *v)
n_st++;
if (!CanVehicleUseStation(v, st)) {
- problem_type = 3;
+ message = STR_NEWS_VEHICLE_HAS_INVALID_ENTRY;
} else if (v->type == VEH_AIRCRAFT &&
(AircraftVehInfo(v->engine_type)->subtype & AIR_FAST) &&
(st->airport.GetFTA()->flags & AirportFTAClass::SHORT_STRIP) &&
_settings_game.vehicle.plane_crashes != 0 &&
!_cheats.no_jetcrash.value &&
- problem_type == -1) {
- problem_type = 4;
+ message == INVALID_STRING_ID) {
+ message = STR_NEWS_PLANE_USES_TOO_SHORT_RUNWAY;
}
}
}
@@ -1808,22 +1807,19 @@ void CheckOrders(const Vehicle *v)
const Order *last = v->GetLastOrder();
if (v->orders.list->GetFirstOrder()->Equals(*last)) {
- problem_type = 2;
+ message = STR_NEWS_VEHICLE_HAS_DUPLICATE_ENTRY;
}
}
/* Do we only have 1 station in our order list? */
- if (n_st < 2 && problem_type == -1) problem_type = 0;
+ if (n_st < 2 && message == INVALID_STRING_ID) message = STR_NEWS_VEHICLE_HAS_TOO_FEW_ORDERS;
#ifndef NDEBUG
if (v->orders.list != NULL) v->orders.list->DebugCheckSanity();
#endif
/* We don't have a problem */
- if (problem_type < 0) return;
-
- message = STR_NEWS_VEHICLE_HAS_TOO_FEW_ORDERS + problem_type;
- //DEBUG(misc, 3, "Triggered News Item for vehicle %d", v->index);
+ if (message == INVALID_STRING_ID) return;
SetDParam(0, v->index);
AddVehicleAdviceNewsItem(message, v->index);