From 27fc22a76bf470c2088a38d57d9cd2a8e481cafb Mon Sep 17 00:00:00 2001
From: yexo <yexo@openttd.org>
Date: Mon, 4 Jan 2010 18:50:50 +0000
Subject: [PATCH] (svn r18724) -Fix [FS#3475] (r17899): refitting a
 non-refittable vehicle to it's default cargotype failed, causing problems for
 AIs

---
 src/vehicle_cmd.cpp | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/vehicle_cmd.cpp b/src/vehicle_cmd.cpp
index f696d11532..a28436654d 100644
--- a/src/vehicle_cmd.cpp
+++ b/src/vehicle_cmd.cpp
@@ -307,13 +307,11 @@ CommandCost RefitVehicle(Vehicle *v, bool only_this, CargoID new_cid, byte new_s
 {
 	CommandCost cost(v->GetExpenseType(false));
 	uint total_capacity = 0;
-	bool success = false;
 
 	v->InvalidateNewGRFCacheOfChain();
 	for (; v != NULL; v = (only_this ? NULL : v->Next())) {
 		const Engine *e = Engine::Get(v->engine_type);
 		if (!e->CanCarryCargo() || !HasBit(e->info.refit_mask, new_cid)) continue;
-		success = true;
 
 		/* Back up the vehicle's cargo type */
 		CargoID temp_cid = v->cargo_type;
@@ -347,7 +345,7 @@ CommandCost RefitVehicle(Vehicle *v, bool only_this, CargoID new_cid, byte new_s
 	}
 
 	_returned_refit_capacity = total_capacity;
-	return success ? cost : CMD_ERROR;
+	return cost;
 }
 
 /** Test if a name is unique among vehicle names.