veitR/mod-playerbots
1
0
Fork 0
mirror of https://github.com/liyunfan1223/mod-playerbots.git synced 2026-06-20 15:39:25 +02:00
Code Issues Packages Projects Releases Wiki Activity

Revert "fix(Core/Travel): Drop short-distance short-circuit in GetFullPath"

Browse Source 
This reverts commit 71dcd6ff09c4cb1975500e3ce5877dbb6018ca91.
This commit is contained in:
bash 2026-05-09 14:15:21 +02:00
parent 1cdc2ec377
commit 999c5343fa
1 changed files with 8 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/Mgr/Travel/TravelNode.cpp
View File

@ -1290,18 +1290,14 @@ bool TravelNodeMap::GetFullPath(TravelPlan& plan,
plan.Reset();
plan.destination = destination;
// Removed the "short distance — direct walk" short-circuit that
// returned a 2-point plan [botPos, destination] without any
// navmesh validation. NODE_PREPATH dispatch then MoveTo'd the
// destination as a single straight-line spline, which clipped
// through walls/mountains whenever bot and dest sat on opposite
// sides of geometry within 296y.
//
// Falling through to the graph build here means: if a graph
// route exists, use it (now validated by IsPathCheating in
// BuildPath + RefineWalkPoints at runtime); otherwise return
// false so MoveFarTo falls through to its chained mmap probe,
// which resolves an mmap-clean waypoint chain.
// Short distance — direct walk, no nodes needed
if (botPos.fDist(destination) < MAX_PATHFINDING_DISTANCE &&
botPos.GetMapId() == destination.GetMapId())
{
plan.steps.addPoint(botPos, PathNodeType::NODE_PREPATH);
plan.steps.addPoint(destination, PathNodeType::NODE_PATH);
return true;
}
std::shared_lock<std::shared_timed_mutex> guard(m_nMapMtx);