From f45221f2745df7ad7bd035bec91c5be808300b70 Mon Sep 17 00:00:00 2001
From: smatz <smatz@openttd.org>
Date: Wed, 19 Mar 2008 20:42:05 +0000
Subject: [PATCH] (svn r12384) -Fix (r2428): do not disconnect train when
 reversing, it may 'think' it is whole in a depot

---
 src/train_cmd.cpp | 22 +++++++---------------
 1 file changed, 7 insertions(+), 15 deletions(-)

diff --git a/src/train_cmd.cpp b/src/train_cmd.cpp
index 763a7ed961..643828a4bc 100644
--- a/src/train_cmd.cpp
+++ b/src/train_cmd.cpp
@@ -52,7 +52,7 @@
 #include "table/train_cmd.h"
 
 static bool TrainCheckIfLineEnds(Vehicle *v);
-static void TrainController(Vehicle *v, bool update_image);
+static void TrainController(Vehicle *v, Vehicle *nomove, bool update_image);
 static TileIndex TrainApproachingCrossingTile(const Vehicle *v);
 
 static const byte _vehicle_initial_x_fract[4] = {10, 8, 4,  8};
@@ -1735,17 +1735,9 @@ static void AdvanceWagons(Vehicle *v, bool before)
 		int differential = last->u.rail.cached_veh_length - base->u.rail.cached_veh_length;
 		if (before) differential *= -1;
 
-		if (differential > 0) {
-			/* disconnect last car to make sure only this subset moves */
-			Vehicle *tempnext = last->Next();
-			last->SetNext(NULL);
-
-			/* do not update images now because the wagons are disconnected
-			 * and that could cause problems with NewGRFs */
-			for (int i = 0; i < differential; i++) TrainController(first, false);
-
-			last->SetNext(tempnext);
-		}
+		/* do not update images now
+		 * negative differential will are handled in the second run */
+		for (int i = 0; i < differential; i++) TrainController(first, last->Next(), false);
 
 		base = first;
 		first = first->Next();
@@ -2993,12 +2985,12 @@ static void *CheckVehicleAtSignal(Vehicle *v, void *data)
 	return NULL;
 }
 
-static void TrainController(Vehicle *v, bool update_image)
+static void TrainController(Vehicle *v, Vehicle *nomove, bool update_image)
 {
 	Vehicle *prev;
 
 	/* For every vehicle after and including the given vehicle */
-	for (prev = v->Previous(); v != NULL; prev = v, v = v->Next()) {
+	for (prev = v->Previous(); v != nomove; prev = v, v = v->Next()) {
 		DiagDirection enterdir = DIAGDIR_BEGIN;
 		bool update_signals_crossing = false; // will we update signals or crossing state?
 		BeginVehicleMove(v);
@@ -3554,7 +3546,7 @@ static void TrainLocoHandler(Vehicle *v, bool mode)
 		TrainCheckIfLineEnds(v);
 
 		do {
-			TrainController(v, true);
+			TrainController(v, NULL, true);
 			CheckTrainCollision(v);
 			if (v->cur_speed <= 0x100)
 				break;