cpp/openttd-patchpack/source Changeset - r3002:026ddefdb659

Changeset - r3002:026ddefdb659

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

tron - 19 years ago 2006-02-09 06:33:37
tron@openttd.org

(svn r3582) Restructure MoveGoodsToStation() to avoid double bookkeeping of StationIDs and station pointers

1 file changed with 22 insertions and 29 deletions:

station_cmd.c

0 comments (0 inline, 0 general)

station_cmd.c

➞

Show inline comments

@@ @@ -2527,9 +2527,8 @@ int32 CmdRenameStation(int x, int y, uin @@
 uint MoveGoodsToStation(TileIndex tile, int w, int h, int type, uint amount)
+{
 	Station *around_ptr[8];
 	StationID around[8];
 	int i;
 	Station* around[8];
 	uint i;
 	uint moved;
 	uint best_rating, best_rating2;
 	Station *st1, *st2;
@@ @@ -2539,8 +2538,7 @@ uint MoveGoodsToStation(TileIndex tile, @@
 	int h_prod;
 	int max_rad;
 	memset(around, 0xff, sizeof(around));
 	for (i = 0; i < lengthof(around); i++) around[i] = NULL;
 	if (_patches.modified_catchment) {
 		w_prod = w;
@@ @@ -2557,17 +2555,15 @@ uint MoveGoodsToStation(TileIndex tile, @@
+	}
 	BEGIN_TILE_LOOP(cur_tile, w, h, tile - TileDiffXY(max_rad, max_rad))
-		StationID st_index;
+		Station* st;
 		cur_tile = TILE_MASK(cur_tile);
 		if (!IsTileType(cur_tile, MP_STATION)) continue;
 		st_index = _m[cur_tile].m2;
 		for (i = 0; i != 8; i++) {
 			if (around[i] == INVALID_STATION) {
 				Station* st = GetStation(st_index);
 		st = GetStation(_m[cur_tile].m2);
 		for (i = 0; i != lengthof(around); i++) {
 			if (around[i] == NULL) {
 				if (!IsBuoy(st) &&
 						(st->town->exclusive_counter == 0 || st->town->exclusivity == st->owner) && // check exclusive transport rights
 						st->goods[type].rating != 0 &&
@@ @@ -2604,25 +2600,22 @@ uint MoveGoodsToStation(TileIndex tile, @@
 						y_dist = 0;
+					}
 					if (x_dist <= rad && y_dist <= rad) {
 						around[i] = st_index;
 						around_ptr[i] = st;
+					}
 					if (x_dist <= rad && y_dist <= rad) around[i] = st;
+				}
 				break;
-			} else if (around[i] == st_index) {
+			} else if (around[i] == st) {
 				break;
+			}
+		}
 	END_TILE_LOOP(cur_tile, w, h, tile - TileDiffXY(max_rad, max_rad))
 	/* no stations around at all? */
 	if (around[0] == INVALID_STATION) return 0;
 	if (around[1] == INVALID_STATION) {
 	if (around[0] == NULL) return 0;
 	if (around[1] == NULL) {
 		/* only one station around */
 		moved = (amount * around_ptr[0]->goods[type].rating >> 8) + 1;
 		UpdateStationWaiting(around_ptr[0], type, moved);
 		moved = (amount * around[0]->goods[type].rating >> 8) + 1;
 		UpdateStationWaiting(around[0], type, moved);
 		return moved;
+	}
@@ @@ -2630,16 +2623,16 @@ uint MoveGoodsToStation(TileIndex tile, @@
 	st2 = st1 = NULL;
 	best_rating = best_rating2 = 0;
 	for (i = 0; i != 8 && around[i] != INVALID_STATION; i++) {
 		if (around_ptr[i]->goods[type].rating >= best_rating) {
 	for (i = 0; i != lengthof(around) && around[i] != NULL; i++) {
 		if (around[i]->goods[type].rating >= best_rating) {
 			best_rating2 = best_rating;
 			st2 = st1;
 			best_rating = around_ptr[i]->goods[type].rating;
 			st1 = around_ptr[i];
 		} else if (around_ptr[i]->goods[type].rating >= best_rating2) {
 			best_rating2 = around_ptr[i]->goods[type].rating;
 			st2 = around_ptr[i];
 			best_rating = around[i]->goods[type].rating;
 			st1 = around[i];
 		} else if (around[i]->goods[type].rating >= best_rating2) {
 			best_rating2 = around[i]->goods[type].rating;
 			st2 = around[i];
+		}
+	}

0 comments (0 inline, 0 general)