fix(Core/Travel): Drop dead future field, restore default copy on LastMovement

This commit is contained in:
bash 2026-05-10 03:22:27 +02:00
parent c64ac89924
commit 9d3f47b19f

View File

@ -28,10 +28,8 @@ class LastMovement
public: public:
LastMovement(); LastMovement();
// Non-copyable: holds a std::future and several state fields the LastMovement(LastMovement const&) = default;
// historical copy-helpers omitted. Always pass by reference. LastMovement& operator=(LastMovement const&) = default;
LastMovement(LastMovement const&) = delete;
LastMovement& operator=(LastMovement const&) = delete;
void clear(); void clear();
@ -57,7 +55,6 @@ public:
MovementPriority priority; MovementPriority priority;
TravelPath lastPath; TravelPath lastPath;
time_t nextTeleport; time_t nextTeleport;
std::future<TravelPath> future;
}; };
class LastMovementValue : public ManualSetValue<LastMovement&> class LastMovementValue : public ManualSetValue<LastMovement&>
@ -65,11 +62,6 @@ class LastMovementValue : public ManualSetValue<LastMovement&>
public: public:
LastMovementValue(PlayerbotAI* botAI) : ManualSetValue<LastMovement&>(botAI, data) {} LastMovementValue(PlayerbotAI* botAI) : ManualSetValue<LastMovement&>(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: private:
LastMovement data{}; LastMovement data{};
}; };