(svn r7386) -Codechange r7385: moved deletion of the vehicle highlight from DeleteVehicle to the sell commands as they are not called as often

   Also added a return to the window loop prevent looking at the rest of the windows once the right depot window is found

1 files changed, 4 insertions, 0 deletions

diff --git a/train_cmd.c b/train_cmd.c
index 69ae01daa..a1792deb5 100644
--- a/train_cmd.c
+++ b/train_cmd.c
@@ -1373,6 +1373,7 @@ int32 CmdSellRailWagon(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
 				cost -= rear->value;
 				if (flags & DC_EXEC) {
 					UnlinkWagon(rear, first);
+					DeleteDepotHighlightOfVehicle(rear);
 					DeleteVehicle(rear);
 				}
 			}
@@ -1417,6 +1418,7 @@ int32 CmdSellRailWagon(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
 			cost -= v->value;
 			if (flags & DC_EXEC) {
 				first = UnlinkWagon(v, first);
+				DeleteDepotHighlightOfVehicle(v);
 				DeleteVehicle(v);
 
 				/* 4 If the second wagon was an engine, update it to front_engine
@@ -1463,6 +1465,7 @@ int32 CmdSellRailWagon(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
 							cost -= rear->value;
 							if (flags & DC_EXEC) {
 								first = UnlinkWagon(rear, first);
+								DeleteDepotHighlightOfVehicle(rear);
 								DeleteVehicle(rear);
 							}
 						}
@@ -1475,6 +1478,7 @@ int32 CmdSellRailWagon(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
 				cost -= v->value;
 				if (flags & DC_EXEC) {
 					first = UnlinkWagon(v, first);
+					DeleteDepotHighlightOfVehicle(v);
 					DeleteVehicle(v);
 				}
 			}