dave/openttd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: Do not send vehicles towards incomplete PF nodes

Browse Source 
YAPF could end up in a situation where it sets the best intermediate node
to a node whose construction is never finalized (i.e. it is never added to
the open list). The content of the node would be overwritten in the next
round, potentially sending the vehicle to an unwanted location.

(cherry picked from commit 0125ba82e8)
This commit is contained in:
Vít Šefl
2021-05-17 22:32:30 +02:00
committed by Jonathan G Rennison
parent ab74800b8e
commit a73ddf4a35
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/pathfinder/yapf/yapf_base.hpp
View File

@@ -260,9 +260,9 @@ public:
return;
}
if (m_max_search_nodes > 0 && (m_pBestIntermediateNode == nullptr || (m_pBestIntermediateNode->GetCostEstimate() - m_pBestIntermediateNode->GetCost()) > (n.GetCostEstimate() - n.GetCost()))) {
m_pBestIntermediateNode = &n;
}
/* The new node can be set as the best intermediate node only once we're
* certain it will be finalized by being inserted into the open list. */
bool set_intermediate = m_max_search_nodes > 0 && (m_pBestIntermediateNode == nullptr || (m_pBestIntermediateNode->GetCostEstimate() - m_pBestIntermediateNode->GetCost()) > (n.GetCostEstimate() - n.GetCost()));
/* check new node against open list */
Node *openNode = m_nodes.FindOpenNode(n.GetKey());
@@ -275,6 +275,7 @@ public:
*openNode = n;
/* add the updated old node back to open list */
m_nodes.InsertOpenNode(*openNode);
if (set_intermediate) m_pBestIntermediateNode = openNode;
}
return;
}
@@ -300,6 +301,7 @@ public:
/* the new node is really new
* add it to the open list */
m_nodes.InsertOpenNode(n);
if (set_intermediate) m_pBestIntermediateNode = &n;
}
const VehicleType * GetVehicle() const