diff options
| author | KUDr <KUDr@openttd.org> | 2007-06-18 14:40:07 +0000 |
|---|---|---|
| committer | KUDr <KUDr@openttd.org> | 2007-06-18 14:40:07 +0000 |
| commit | 80d71953ba2bbf49157ec1208589e295bb900def (patch) | |
| tree | cc8f707ad77f07d941100d315c6884eaa2ed293a /src/yapf | |
| parent | 5718743722122a7a9f53925b5a5a015b3a9e16cb (diff) | |
| download | openttd-80d71953ba2bbf49157ec1208589e295bb900def.tar.xz | |
(svn r10199) -Fix [FS#884, YAPF]: 'target_seen' flag that is set prematurely in some cases (1 tile long cached segment followed by target station) stops on assert (immeR)
Diffstat (limited to 'src/yapf')
| -rw-r--r-- | src/yapf/yapf_costrail.hpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/yapf/yapf_costrail.hpp b/src/yapf/yapf_costrail.hpp index ffde3703b..9dc2cc701 100644 --- a/src/yapf/yapf_costrail.hpp +++ b/src/yapf/yapf_costrail.hpp @@ -195,6 +195,7 @@ public: RailType rail_type = GetTileRailType(tile); bool target_seen = Yapf().PfDetectDestination(tile, trackdir); + bool end_by_target_seen = false; if (tf.m_is_station) { // station tiles have an extra penalty @@ -212,6 +213,7 @@ public: // finish if we have reached the destination if (target_seen) { + end_by_target_seen = true; break; } @@ -342,7 +344,7 @@ public: } // special costs for the case we have reached our target - if (target_seen) { + if (end_by_target_seen) { n.flags_u.flags_s.m_targed_seen = true; if (n.flags_u.flags_s.m_last_signal_was_red) { if (n.m_last_red_signal_type == SIGTYPE_EXIT) { @@ -358,7 +360,7 @@ public: // total node cost n.m_cost = parent_cost + first_tile_cost + segment_cost + extra_cost; - return !n.m_segment->flags_u.flags_s.m_end_of_line; + return !n.m_segment->flags_u.flags_s.m_end_of_line || end_by_target_seen; } FORCEINLINE bool CanUseGlobalCache(Node& n) const |