[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)
X:25 = sql for sdX2 commit
* DK starting area quest -
* Quest how to win friends and how to influence enemies
* Few more npc updates to ebon hold mobs
  Taking a closer look my second time around complete testing
[mr3067] More accurate reserving memory for internal container in WorldPacket
[mr3068] Implement and use Unit::IsDamageReducedByArmor()

Pried in TC.
[mr3069] Always send compressed update packet if his size less than uncompressed

Decrease net traffic.
Fix compile
[mr3070] move one lock to another place (by valgrind check result).
[mr3071] small rework in Map::SendUpdates (pick from ZoneUpdating).