From a18e9ccf27abbe5fb35ee0a8c577c7e51a205779 Mon Sep 17 00:00:00 2001 From: celestar Date: Sat, 22 Apr 2006 05:36:22 +0000 Subject: [PATCH] (svn r4515) -Codechange: TPFMode2 is currently only used for TRANSPORT_WATER. So remove all stuff that deals with other transport types and assert TRANSPORT_WATER --- Makefile | 2 +- pathfind.c | 35 +---------------------------------- 2 files changed, 2 insertions(+), 35 deletions(-) diff --git a/Makefile b/Makefile index 414421822e..e284672bda 100644 --- a/Makefile +++ b/Makefile @@ -809,7 +809,7 @@ ifndef NATIVE_OSX # OSX links in os/macosx/Makefile to handle universal binaries better $(TTD): $(OBJS) $(MAKE_CONFIG) @echo '===> Linking $@' - $(Q)$(CC) $(LDFLAGS) $(TTDLDFLAGS) $(OBJS) $(LIBS) -o $@ + $(Q)$(CC) $(LDFLAGS) -lefence $(TTDLDFLAGS) $(OBJS) $(LIBS) -o $@ endif $(STRGEN): strgen/strgen.c endian_host.h diff --git a/pathfind.c b/pathfind.c index e3a21398d5..3592c372df 100644 --- a/pathfind.c +++ b/pathfind.c @@ -134,47 +134,14 @@ static void TPFMode2(TrackPathFinder* tpf, TileIndex tile, DiagDirection directi uint bits; int i; RememberData rd; - int owner = -1; - /* XXX: Mode 2 is currently only used for ships, why is this code here? */ - if (tpf->tracktype == TRANSPORT_RAIL) { - switch (GetTileType(tile)) { - case MP_TUNNELBRIDGE: - // bridge middle has no owner - if (IsBridge(tile) && IsBridgeMiddle(tile)) break; - /* FALLTHROUGH */ - - case MP_RAILWAY: - case MP_STATION: - owner = GetTileOwner(tile); - break; - - default: break; // XXX can this occur? - } - } + assert(tpf->tracktype == TRANSPORT_WATER); // This addition will sometimes overflow by a single tile. // The use of TILE_MASK here makes sure that we still point at a valid // tile, and then this tile will be in the sentinel row/col, so GetTileTrackStatus will fail. tile = TILE_MASK(tile + TileOffsByDir(direction)); - /* Check in case of rail if the owner is the same */ - if (tpf->tracktype == TRANSPORT_RAIL) { - switch (GetTileType(tile)) { - case MP_TUNNELBRIDGE: - // bridge middle has no owner - if (IsBridge(tile) && IsBridgeMiddle(tile)) break; - /* FALLTHROUGH */ - - case MP_RAILWAY: - case MP_STATION: - if (owner != -1 && !IsTileOwner(tile, owner)) return; - break; - - default: break; // XXX can this occur? - } - } - if (++tpf->rd.cur_length > 50) return;