-Crash fix: Stop random crashes with tweakvalue enabled.

celguar · Jul 11, 2023 · 522f482 · 522f482
1 parent 0c23a5f
commit 522f482
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 17 deletions.
diff --git a/playerbot/strategy/actions/MovementActions.cpp b/playerbot/strategy/actions/MovementActions.cpp
@@ -418,22 +418,7 @@ bool MovementAction::MoveTo(uint32 mapId, float x, float y, float z, bool idle,
         {
             if (!sTravelNodeMap.getNodes().empty() && !bot->InBattleGround())
             {
-                if (sPlayerbotAIConfig.tweakValue)
-                {
-                    if (lastMove.future.valid())
-                    {
-                        if(lastMove.future.wait_for(0s) != future_status::ready)
-                            return true; //we have no path. Wait until it is ready.
-                        movePath = lastMove.future.get();
-                    }
-                    else
-                    {
-                        lastMove.future = std::async(&TravelNodeMap::getFullPath, startPosition, endPosition, bot);
-                        return true;
-                    }
-                }
-                else
-                    movePath = sTravelNodeMap.getFullPath(startPosition, endPosition, bot);
+                movePath = sTravelNodeMap.getFullPath(startPosition, endPosition, bot);
 
                 if (movePath.empty())
                 {

diff --git a/playerbot/strategy/values/LastMovementValue.h b/playerbot/strategy/values/LastMovementValue.h
@@ -99,7 +99,6 @@ namespace ai
         WorldPosition lastMoveShort;
         TravelPath lastPath;
         time_t nextTeleport;
-        std::future<TravelPath> future;
     };
 
     class LastMovementValue : public ManualSetValue<LastMovement&>