diff options
author | bjarni <bjarni@openttd.org> | 2008-01-19 23:34:52 +0000 |
---|---|---|
committer | bjarni <bjarni@openttd.org> | 2008-01-19 23:34:52 +0000 |
commit | e91230415d37882176da2e9bf5f863779b3db745 (patch) | |
tree | 7afaef32fd8a76ab03791943c4a87e2a8a5d3413 /src | |
parent | 2f6ee3a4a119fbddb7dcde34c8ceaf5ac63efb14 (diff) | |
download | openttd-e91230415d37882176da2e9bf5f863779b3db745.tar.xz |
(svn r11929) -Fix (r9981)[FS#1624]: [autoreplace] fixed a case where a single headed locomotive caused an assert when being replaced to a dualheaded one
This triggered if the player had enough money to replace and refit one of them but not enough to refit the last one as well
Diffstat (limited to 'src')
-rw-r--r-- | src/autoreplace_cmd.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/autoreplace_cmd.cpp b/src/autoreplace_cmd.cpp index ef5e0d70f..b451be8de 100644 --- a/src/autoreplace_cmd.cpp +++ b/src/autoreplace_cmd.cpp @@ -21,6 +21,7 @@ #include "functions.h" #include "variables.h" #include "autoreplace_func.h" +#include "articulated_vehicles.h" #include "table/strings.h" @@ -175,7 +176,10 @@ static CommandCost ReplaceVehicle(Vehicle **w, byte flags, Money total_cost) if (replacement_cargo_type != CT_NO_REFIT) { /* add refit cost */ CommandCost refit_cost = GetRefitCost(new_engine_type); - if (old_v->type == VEH_TRAIN && IsMultiheaded(old_v)) refit_cost.AddCost(refit_cost); // pay for both ends + if (old_v->type == VEH_TRAIN && RailVehInfo(new_engine_type)->railveh_type == RAILVEH_MULTIHEAD) { + /* Since it's a dualheaded engine we have to pay once more because the rear end is being refitted too. */ + refit_cost.AddCost(refit_cost); + } cost.AddCost(refit_cost); } |