From 9d3f47b19f1531e969ddee0b3f1f9893b67ff793 Mon Sep 17 00:00:00 2001 From: bash Date: Sun, 10 May 2026 03:22:27 +0200 Subject: [PATCH] fix(Core/Travel): Drop dead future field, restore default copy on LastMovement --- src/Ai/Base/Value/LastMovementValue.h | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/src/Ai/Base/Value/LastMovementValue.h b/src/Ai/Base/Value/LastMovementValue.h index 0827bb26e..d8cbf9471 100644 --- a/src/Ai/Base/Value/LastMovementValue.h +++ b/src/Ai/Base/Value/LastMovementValue.h @@ -28,10 +28,8 @@ class LastMovement public: LastMovement(); - // Non-copyable: holds a std::future and several state fields the - // historical copy-helpers omitted. Always pass by reference. - LastMovement(LastMovement const&) = delete; - LastMovement& operator=(LastMovement const&) = delete; + LastMovement(LastMovement const&) = default; + LastMovement& operator=(LastMovement const&) = default; void clear(); @@ -57,7 +55,6 @@ public: MovementPriority priority; TravelPath lastPath; time_t nextTeleport; - std::future future; }; class LastMovementValue : public ManualSetValue @@ -65,11 +62,6 @@ class LastMovementValue : public ManualSetValue public: LastMovementValue(PlayerbotAI* botAI) : ManualSetValue(botAI, data) {} - // LastMovement is non-copyable; the base Reset/Set rely on - // operator= which we deleted. Override both to do the right thing. - void Reset() override { data.clear(); } - void Set(LastMovement& /*val*/) override {} - private: LastMovement data{}; };