Mirror/TrinityCore
1
0
Fork 0
mirror of https://github.com/TrinityCore/TrinityCore.git synced 2026-01-15 23:20:36 +01:00
Code Issues Packages Projects Releases Wiki Activity

Core/Movement: remove redundant method MovementGenerator::NotifyAIOnFinalize and use existing Finalize - MovementInform implementation

Browse Source 
6fde051e4e follow up
This commit is contained in:
ccrs
2021-06-03 15:01:47 +02:00
parent e7e4786606
commit 303e01c834
7 changed files with 10 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/server/game/AI/CoreAI/PassiveAI.cpp
View File

@@ -79,7 +79,7 @@ void CritterAI::JustEngagedWith(Unit* /*who*/)
me->SetControlled(true, UNIT_STATE_FLEEING);
}
void CritterAI::OnMovementGeneratorFinalized(MovementGeneratorType type)
void CritterAI::MovementInform(uint32 type, uint32 id)
{
if (type == TIMED_FLEEING_MOTION_TYPE)
EnterEvadeMode(EVADE_REASON_OTHER);

2
src/server/game/AI/CoreAI/PassiveAI.h
View File

@@ -75,7 +75,7 @@ class TC_GAME_API CritterAI : public PassiveAI
void JustEngagedWith(Unit* /*who*/) override;
void EnterEvadeMode(EvadeReason why) override;
void OnMovementGeneratorFinalized(MovementGeneratorType type) override;
void MovementInform(uint32 type, uint32 id) override;
static int32 Permissible(Creature const* creature);
};

2
src/server/game/AI/CoreAI/UnitAI.h
View File

@@ -334,8 +334,6 @@ class TC_GAME_API UnitAI
// Called when a game event starts or ends
virtual void OnGameEvent(bool /*start*/, uint16 /*eventId*/) { }
virtual void OnMovementGeneratorFinalized(MovementGeneratorType /*type*/) { }
virtual std::string GetDebugInfo() const;
private:

5
src/server/game/Movement/MotionMaster.cpp
View File

@@ -1148,10 +1148,7 @@ void MotionMaster::DirectAdd(MovementGenerator* movement, MovementSlot slot/* =
{
case MOTION_SLOT_DEFAULT:
if (_defaultGenerator)
{
_defaultGenerator->Finalize(_owner, _generators.empty(), false);
_defaultGenerator->NotifyAIOnFinalize(_owner);
}
_defaultGenerator = MovementGeneratorPointer(movement);
if (IsStatic(movement))
@@ -1196,7 +1193,6 @@ void MotionMaster::Delete(MovementGenerator* movement, bool active, bool movemen
movement->Priority, movement->Flags, movement->BaseUnitState, movement->GetMovementGeneratorType(), _owner->GetGUID().ToString().c_str());
movement->Finalize(_owner, active, movementInform);
movement->NotifyAIOnFinalize(_owner);
ClearBaseUnitState(movement);
MovementGeneratorPointerDeleter(movement);
}
@@ -1207,7 +1203,6 @@ void MotionMaster::DeleteDefault(bool active, bool movementInform)
_defaultGenerator->Priority, _defaultGenerator->Flags, _defaultGenerator->BaseUnitState, _defaultGenerator->GetMovementGeneratorType(), _owner->GetGUID().ToString().c_str());
_defaultGenerator->Finalize(_owner, active, movementInform);
_defaultGenerator->NotifyAIOnFinalize(_owner);
_defaultGenerator = MovementGeneratorPointer(GetIdleMovementGenerator());
AddFlag(MOTIONMASTER_FLAG_STATIC_INITIALIZATION_PENDING);
}

6
src/server/game/Movement/MovementGenerator.cpp
View File

@@ -58,9 +58,3 @@ MovementGenerator* WaypointMovementFactory::Create(Unit* /*object*/) const
{
return new WaypointMovementGenerator<Creature>();
}
void MovementGenerator::NotifyAIOnFinalize(Unit* object)
{
if (UnitAI* ai = object->GetAI())
ai->OnMovementGeneratorFinalized(GetMovementGeneratorType());
}

2
src/server/game/Movement/MovementGenerator.h
View File

@@ -69,8 +69,6 @@ class TC_GAME_API MovementGenerator
// used by Evade code for select point to evade with expected restart default movement
virtual bool GetResetPosition(Unit*, float&/* x*/, float&/* y*/, float&/* z*/) { return false; }
virtual void NotifyAIOnFinalize(Unit*);
void AddFlag(uint16 const flag) { Flags |= flag; }
bool HasFlag(uint16 const flag) const { return (Flags & flag) != 0; }
void RemoveFlag(uint16 const flag) { Flags &= ~flag; }

9
src/server/game/Movement/MovementGenerators/FleeingMovementGenerator.cpp
View File

@@ -250,7 +250,7 @@ bool TimedFleeingMovementGenerator::Update(Unit* owner, uint32 diff)
return FleeingMovementGenerator<Creature>::DoUpdate(owner->ToCreature(), diff);
}
void TimedFleeingMovementGenerator::Finalize(Unit* owner, bool active, bool/* movementInform*/)
void TimedFleeingMovementGenerator::Finalize(Unit* owner, bool active, bool movementInform)
{
AddFlag(MOVEMENTGENERATOR_FLAG_FINALIZED);
if (!active)
@@ -266,6 +266,13 @@ void TimedFleeingMovementGenerator::Finalize(Unit* owner, bool active, bool/* mo
owner->ToCreature()->AI()->AttackStart(victim);
}
}
if (movementInform)
{
Creature* ownerCreature = owner->ToCreature();
if (CreatureAI* AI = ownerCreature ? ownerCreature->AI() : nullptr)
AI->MovementInform(TIMED_FLEEING_MOTION_TYPE, 0);
}
}
MovementGeneratorType TimedFleeingMovementGenerator::GetMovementGeneratorType() const