cpp/openttd-patchpack/source Changeset - r3384:cd5f7becd5fc

Changeset - r3384:cd5f7becd5fc

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

peter1138 - 18 years ago 2006-03-31 08:39:10
peter1138@openttd.org

(svn r4192) - Fix: In the depot, moving wagons from the end of a very long train to a short train where the short train was scrolled off the window would cause a game crash. (Thanks to SimonRC for reporting.)

1 file changed with 2 insertions and 5 deletions:

train_gui.c

0 comments (0 inline, 0 general)

train_gui.c

➞

Show inline comments

@@ @@ -515,13 +515,10 @@ found_it: @@
 	if (x < 0 && IsFrontEngine(v)) return (x >= -10) ? -2 : -1;
 	// skip vehicles that are scrolled off the left side
 	while (skip--) v = v->next;
+	while (v != NULL && skip--) v = v->next;
 	/* find the vehicle in this row that was clicked */
 	while ((x -= WagonLengthToPixels(v->u.rail.cached_veh_length)) >= 0) {
 		v = v->next;
 		if (v == NULL) break;
+	}
 	while (v != NULL && (x -= WagonLengthToPixels(v->u.rail.cached_veh_length)) >= 0) v = v->next;
 	// if an articulated part was selected, find its parent
 	while (v != NULL && IsArticulatedPart(v)) v = GetPrevVehicleInChain(v);

0 comments (0 inline, 0 general)