From d0cdb45b6c634b4b7238f9b8b8cd9344cb7b1ac3 Mon Sep 17 00:00:00 2001
From: hackykid <hackykid@openttd.org>
Date: Tue, 7 Jun 2005 16:44:54 +0000
Subject: [PATCH] (svn r2431)  - Fix: [newgrf] clamp() instead of assert() on
 the result of the vehicle length calculation, as it can depend on external
 input. (thanks tron)

---
 train_cmd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/train_cmd.c b/train_cmd.c
index 74dd5bf394..b12b49de6b 100644
--- a/train_cmd.c
+++ b/train_cmd.c
@@ -119,7 +119,7 @@ void TrainConsistChanged(Vehicle *v) {
 			veh_len = GetCallBackResult(CBID_VEH_LENGTH,  u->engine_type, u);
 		if (veh_len == CALLBACK_FAILED)
 			veh_len = rvi_u->shorten_factor;
-		assert(veh_len < 8);
+		veh_len = clamp(veh_len, 0, 7);
 		u->u.rail.cached_veh_length = 8 - veh_len;
 
 	};