[mr3056] Player: some improvements and code style
Revert "[mr3056] Player: some improvements and code style"

This reverts commit b1684395530776b996fd53661b151a49a9e17b61.
(HUGE count of compilation errors, impossible compile. please check compilation BEFORE commit)
[mr3056] Player: code style
[mr3057] Player: improve error output
[mr3058] Player: improve sql statements
[mr3059] Player: remove trailing spaces from the string DB data
[mr3060] Player: rename RandomWinner -> RandomBGWinner
[mr3061] Player: activate spec improvement
[mr3062] Player: misc improvements
[mr3063] Optimize getVictim() usage (speed up things)

We have in Unit:
Unit* getVictim() const { return IsInWorld() ? GetMap()->GetUnit(m_attackingGuid) : NULL; }

In code many calls like:
if (getVictim())
    Do(getVictim());

So, everytime we search in objectsStore container (twice and more) the same unit, what takes time.
For speed need to find a one time only.
[mr3064] Optimize getVictim() function

Really, why need search for empty guid?
[mr3065] PetAI: remove "dead" code and fix pet aggressive behavior
[mr3066] fix compilation for mr3056-3062 (need additional live tests)
Merge commit 'c592286ea54dbf79ed3bde05f090ef5b3b898ef7'

Conflicts:
	src/game/BattleGround/BattleGround.cpp
	src/game/BattleGround/BattleGroundMgr.cpp
	src/game/Object.cpp
	src/game/Player.cpp
	src/game/SpellAuras.cpp
	src/game/TargetedMovementGenerator.cpp