Fix memory leaks in GameEventMgr, MotionMaster, ObjectMgr and Pet

[schell244]

🍰 Pullrequest

Fix some memory leaks and uninitialized member variables discovered during Valgrind profiling.

• GameEventMgr: Implement destructor to clean up mGameEventHardcodedList entries
• MotionMaster: Fix memory leak in ClearType() by deleting non-static movement generators
• ObjectMgr: Add cleanup methods for item prototypes and creature auras
• Pet: Add ClearCharmInfo() call in destructor
• MasterPlayer: Initialize unReadMails and m_nextMailDelivereTime in constructor
• battleground_alterac: Initialize Event_Timer and Point in Reset()
• MotionMaster: Remove stray semicolon
• PlayerBotAI: Add cleanup on error path and PvP flag initialization

Proof

• None

Issues

• None

How2Test

• None

Todo / Checklist

• None