From b05919cc5a07ef01e18db2a22f339fb8a5665526 Mon Sep 17 00:00:00 2001 From: bjarni Date: Tue, 25 Mar 2008 21:58:13 +0000 Subject: (svn r12421) -Feature: [autoreplace] the autoreplace button in train depots will now also replace wagons even if they aren't connected to a locomotive fixed estimated cost in CmdDepotMassAutoReplace() (will still not estimate wagon removal profits) Made it possible to command CmdDepotMassAutoReplace() to either replace everything or nothing (the button will still happily replace just some of the vehicles if cash premits) --- src/command.cpp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'src/command.cpp') diff --git a/src/command.cpp b/src/command.cpp index bb519cbd3..c00a40eaa 100644 --- a/src/command.cpp +++ b/src/command.cpp @@ -542,12 +542,16 @@ bool DoCommandP(TileIndex tile, uint32 p1, uint32 p2, CommandCallback *callback, * fact will trigger an assertion failure. --pasky * CMD_CLONE_VEHICLE: Both building new vehicles and refitting them can be * influenced by newgrf callbacks, which makes it impossible to accurately - * estimate the cost of cloning a vehicle. */ + * estimate the cost of cloning a vehicle. + * CMD_DEPOT_MASS_AUTOREPLACE: we can't predict wagon removal so + * the test will not include income from any sold wagons. + * This means that the costs can sometimes be lower than estimated. */ notest = (cmd & 0xFF) == CMD_CLEAR_AREA || (cmd & 0xFF) == CMD_LEVEL_LAND || (cmd & 0xFF) == CMD_REMOVE_LONG_ROAD || - (cmd & 0xFF) == CMD_CLONE_VEHICLE; + (cmd & 0xFF) == CMD_CLONE_VEHICLE || + (cmd & 0xFF) == CMD_DEPOT_MASS_AUTOREPLACE; _docommand_recursive = 1; -- cgit v1.2.3-70-g09d2