summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorludde <ludde@openttd.org>2005-07-17 13:45:43 +0000
committerludde <ludde@openttd.org>2005-07-17 13:45:43 +0000
commit62011eed95fcd374c2debbc177612e9685926666 (patch)
treef8a39ab587287a85793f44677499e0d854b8921f
parent658f9cb03f9a5ec8a0dd772f6cadf54bf53a627a (diff)
downloadopenttd-62011eed95fcd374c2debbc177612e9685926666.tar.xz
(svn r2603) Fix [multistop] Fix problem with multistop handling, the range in my fix was way too high. Also only lock a slot for 5 days instead of 30.
-rw-r--r--roadveh_cmd.c27
1 files changed, 24 insertions, 3 deletions
diff --git a/roadveh_cmd.c b/roadveh_cmd.c
index 3c5fdfdf9..c79fd9c01 100644
--- a/roadveh_cmd.c
+++ b/roadveh_cmd.c
@@ -1649,7 +1649,7 @@ void OnNewDay_RoadVeh(Vehicle *v)
RoadStop *rs = GetPrimaryRoadStop(st, type);
RoadStop *first_stop = rs;
RoadStop *best_stop = NULL;
- uint32 mindist = 120, dist; // 120 is threshold distance.
+ uint32 mindist = 12, dist; // 12 is threshold distance.
//first we need to find out how far our stations are away.
DEBUG(ms, 2) ("Multistop: Attempting to obtain a slot for vehicle %d at station %d (0x%x)", v->unitnumber, st->index, st->xy);
@@ -1662,6 +1662,27 @@ void OnNewDay_RoadVeh(Vehicle *v)
assert(NUM_SLOTS == 2);
dist = DistanceManhattan(v->tile, rs->xy);
+ // Check if the station is located BEHIND the vehicle..
+ // In that case, add penalty.
+ switch(v->direction) {
+ case 1: // going north east,x position decreasing
+ if (v->x_pos <= TileX(rs->xy) * 16 + 15)
+ dist += 6;
+ break;
+ case 3: // Going south east, y position increasing
+ if (v->y_pos >= TileY(rs->xy) * 16)
+ dist += 6;
+ break;
+ case 5: // Going south west, x position increasing
+ if (v->x_pos >= TileX(rs->xy) * 16)
+ dist += 6;
+ break;
+ case 7: // Going north west, y position decrasing.
+ if (v->y_pos <= TileY(rs->xy) * 16 + 15)
+ dist += 6;
+ break;
+ }
+
// Remember the one with the shortest distance
if (dist < mindist) {
mindist = dist;
@@ -1679,9 +1700,9 @@ void OnNewDay_RoadVeh(Vehicle *v)
best_stop->slot[slot] = v->index;
v->u.road.slot = best_stop;
v->dest_tile = best_stop->xy;
- v->u.road.slot_age = -30;
+ v->u.road.slot_age = -5;
v->u.road.slotindex = slot;
- DEBUG(ms, 1) ("Multistop: Slot %d at 0x%x assigned to vehicle %d", slot, best_stop->xy, v->unitnumber);
+ DEBUG(ms, 1) ("Multistop: Slot %d at 0x%x assigned to vehicle %d (0x%x)", slot, best_stop->xy, v->unitnumber, v->tile);
} else if (first_stop) {
//now we couldn't assign a slot for one reason or another.
//so we just go towards the first station