# HG changeset patch
# User alberth <alberth@openttd.org>
# Date 2010-03-07 16:10:40
# Node ID ffbec4f9cf484947494c4a63ff95a633a8946f01
# Parent  38eb1d96fb0bdab6d1e8d8ac7e9f551bf07530d2

(svn r19366) -Codechange: Move setting of _error_message from the callback check routine to the caller.

diff --git a/src/vehicle.cpp b/src/vehicle.cpp
--- a/src/vehicle.cpp
+++ b/src/vehicle.cpp
@@ -461,7 +461,6 @@ static Vehicle *EnsureNoTrainOnTrackProc
 	Train *t = Train::From(v);
 	if ((t->track != rail_bits) && !TracksOverlap(t->track | rail_bits)) return NULL;
 
-	_error_message = STR_ERROR_TRAIN_IN_THE_WAY + v->type;
 	return v;
 }
 
@@ -475,7 +474,13 @@ static Vehicle *EnsureNoTrainOnTrackProc
  */
 bool EnsureNoTrainOnTrackBits(TileIndex tile, TrackBits track_bits)
 {
-	return !HasVehicleOnPos(tile, &track_bits, &EnsureNoTrainOnTrackProc);
+	/* Value v is not safe in MP games, however, it is used to generate a local
+	 * error message only (which may be different for different machines).
+	 * Such a message does not affect MP synchronisation.
+	 */
+	Vehicle *v = VehicleFromPos(tile, &track_bits, &EnsureNoTrainOnTrackProc, true);
+	if (v != NULL) _error_message = STR_ERROR_TRAIN_IN_THE_WAY + v->type;
+	return v == NULL;
 }
 
 static void UpdateNewVehiclePosHash(Vehicle *v, bool remove)