From 23369337c1af635f479c1d1541601742f6285dd2 Mon Sep 17 00:00:00 2001
From: rubidium <rubidium@openttd.org>
Date: Sat, 11 Dec 2010 13:48:30 +0000
Subject: [PATCH] (svn r21466) -Codechange: make VehicleHasDepotOrders a
 function of Vehicle.

---
 src/order_cmd.cpp  | 11 ++++-------
 src/order_func.h   |  1 -
 src/vehicle.cpp    |  4 ++--
 src/vehicle_base.h |  1 +
 4 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/src/order_cmd.cpp b/src/order_cmd.cpp
index d02522cc6e..13bdf2d8c0 100644
--- a/src/order_cmd.cpp
+++ b/src/order_cmd.cpp
@@ -1445,17 +1445,14 @@ void RemoveOrderFromAllVehicles(OrderType type, DestinationID destination)
 }
 
 /**
- *
- * Checks if a vehicle has a GOTO_DEPOT in his order list
- *
- * @return True if this is true (lol ;))
- *
+ * Checks if a vehicle has a depot in its order list.
+ * @return True iff at least one order is a depot order.
  */
-bool VehicleHasDepotOrders(const Vehicle *v)
+bool Vehicle::HasDepotOrder() const
 {
 	const Order *order;
 
-	FOR_VEHICLE_ORDERS(v, order) {
+	FOR_VEHICLE_ORDERS(this, order) {
 		if (order->IsType(OT_GOTO_DEPOT)) return true;
 	}
 
diff --git a/src/order_func.h b/src/order_func.h
index 5736cb0ca0..638c940af2 100644
--- a/src/order_func.h
+++ b/src/order_func.h
@@ -19,7 +19,6 @@
 /* Functions */
 void RemoveOrderFromAllVehicles(OrderType type, DestinationID destination);
 void InvalidateVehicleOrder(const Vehicle *v, int data);
-bool VehicleHasDepotOrders(const Vehicle *v);
 void CheckOrders(const Vehicle*);
 void DeleteVehicleOrders(Vehicle *v, bool keep_orderlist = false);
 bool ProcessOrders(Vehicle *v);
diff --git a/src/vehicle.cpp b/src/vehicle.cpp
index ff6039c9a1..b7bae6cb09 100644
--- a/src/vehicle.cpp
+++ b/src/vehicle.cpp
@@ -158,8 +158,8 @@ bool Vehicle::NeedsServicing() const
 
 bool Vehicle::NeedsAutomaticServicing() const
 {
-	if (_settings_game.order.gotodepot && VehicleHasDepotOrders(this)) return false;
-	if (this->current_order.IsType(OT_LOADING))            return false;
+	if (_settings_game.order.gotodepot && this->HasDepotOrder()) return false;
+	if (this->current_order.IsType(OT_LOADING)) return false;
 	if (this->current_order.IsType(OT_GOTO_DEPOT) && this->current_order.GetDepotOrderType() != ODTFB_SERVICE) return false;
 	return NeedsServicing();
 }
diff --git a/src/vehicle_base.h b/src/vehicle_base.h
index 20cd9ee211..4dfa4e6f10 100644
--- a/src/vehicle_base.h
+++ b/src/vehicle_base.h
@@ -658,6 +658,7 @@ public:
 	}
 
 	bool IsEngineCountable() const;
+	bool HasDepotOrder() const;
 };
 
 #define FOR_ALL_VEHICLES_FROM(var, start) FOR_ALL_ITEMS_FROM(Vehicle, vehicle_index, var, start)