(svn r1735) -Fix: [ 1112285 ] Fixed an assertion error with stations where stops
authorcelestar
Sun, 30 Jan 2005 17:46:57 +0000
changeset 1231 5fa7581377a9
parent 1230 f08094212688
child 1232 7e5c906c642c
(svn r1735) -Fix: [ 1112285 ] Fixed an assertion error with stations where stops
were deleted, the assertions were just plain wrong at that place. Used a
more graceful handling. Todo: What happens if ALL stations in a schedule
have no suitable stop?
roadveh_cmd.c
station_cmd.c
--- a/roadveh_cmd.c	Sun Jan 30 16:54:39 2005 +0000
+++ b/roadveh_cmd.c	Sun Jan 30 17:46:57 2005 +0000
@@ -628,7 +628,14 @@
 			num = GetNumRoadStops(st, type);
 			rs = GetPrimaryRoadStop(st, type);
 
-			assert (rs != NULL);
+			if (rs == NULL) {
+				int i, validorders = 0;
+				//There is no stop left at the station, so don't even TRY to go there
+				v->cur_order_index++;
+				InvalidateVehicleOrder(v);
+
+				return;
+			}
 
 			dist = malloc(num * sizeof(int32));
 
@@ -1632,7 +1639,8 @@
 		//We do not have a slot, so make one
 		if (v->u.road.slot == NULL) {
 			//first we need to find out how far our stations are away.
-			assert( rs != NULL);
+			if ( rs == NULL )
+				goto no_stop;
 
 			do {
 				stop->dist = 0xFFFFFFFF;
@@ -1688,6 +1696,7 @@
 		firststop = stop = NULL;
 	}
 
+no_stop:
 	if (v->vehstatus & VS_STOPPED)
 		return;
 
--- a/station_cmd.c	Sun Jan 30 16:54:39 2005 +0000
+++ b/station_cmd.c	Sun Jan 30 17:46:57 2005 +0000
@@ -166,7 +166,18 @@
 		case VEH_Train: 		return st->train_tile;
 		case VEH_Aircraft:	return st->airport_tile;
 		case VEH_Ship:			return st->dock_tile;
-		case VEH_Road:			return (v->cargo_type == CT_PASSENGERS) ? st->bus_stops->xy : st->truck_stops->xy;
+		case VEH_Road:
+			if (v->cargo_type == CT_PASSENGERS) {
+				if (st->bus_stops != NULL)
+					return st->bus_stops->xy;
+				else
+					return 0;
+			} else {
+				if (st->truck_stops != NULL)
+					return st->truck_stops->xy;
+				else
+					return 0;
+			}
 		default:
 			assert(false);
 			return 0;