# HG changeset patch # User richk # Date 1174254240 0 # Node ID 8568add6b699b165fca71ec215dc641d9e8f0320 # Parent 2edfed1e6931d03ee951814b9a429548f7a82235 (svn r9299) -Fix (FS#675,FS#660): Small micro-movements on airports (of <4 pixels) caused odd aircraft movements. On oilrigs, this caused a full crash as movement would take helicopter out of airport/station tile. Corrected by maneuvering aircraft directly over these tiny movements, rather than relying on movement by changing facing. diff --git a/src/aircraft_cmd.cpp b/src/aircraft_cmd.cpp --- a/src/aircraft_cmd.cpp +++ b/src/aircraft_cmd.cpp @@ -1063,19 +1063,38 @@ if (v->load_unload_time_rem != 0) v->load_unload_time_rem--; do { - /* Turn. Do it slowly if in the air. */ - Direction newdir = GetDirectionTowards(v, x + amd->x, y + amd->y); - if (newdir != v->direction) { - v->direction = newdir; - if (amd->flag & AMED_SLOWTURN) { - if (v->load_unload_time_rem == 0) v->load_unload_time_rem = 8; - } else { - v->cur_speed >>= 1; + + GetNewVehiclePosResult gp; + + if (dist < 4) { + /* move vehicle one pixel towards target */ + gp.x = (v->x_pos != (x + amd->x)) ? + v->x_pos + ((x + amd->x > v->x_pos) ? 1 : -1) : + v->x_pos; + gp.y = (v->y_pos != (y + amd->y)) ? + v->y_pos + ((y + amd->y > v->y_pos) ? 1 : -1) : + v->y_pos; + + /* Oilrigs must keep v->tile as st->airport_tile, since the landing pad is in a non-airport tile */ + gp.new_tile = (st->airport_type == AT_OILRIG) ? st->airport_tile : TileVirtXY(gp.x, gp.y); + + } else { + + /* Turn. Do it slowly if in the air. */ + Direction newdir = GetDirectionTowards(v, x + amd->x, y + amd->y); + if (newdir != v->direction) { + v->direction = newdir; + if (amd->flag & AMED_SLOWTURN) { + if (v->load_unload_time_rem == 0) v->load_unload_time_rem = 8; + } else { + v->cur_speed >>= 1; + } } + + /* Move vehicle. */ + gp = GetNewVehiclePos(v); } - /* Move vehicle. */ - GetNewVehiclePosResult gp = GetNewVehiclePos(v); v->tile = gp.new_tile; /* If vehicle is in the air, use tile coordinate 0. */ // if (amd->flag & (AMED_TAKEOFF | AMED_SLOWTURN | AMED_LAND)) v->tile = 0;