[PATCH] Nuke charge

[fatherbushido]

As charge is not really designed, according to some discussions on irc, it is perhaps better to nuke the remaining trace of it in the current code. The one who will implement it could do it in the way he wants.

The patch above remove all trace of it (code, templates) and change the current useless maharbal aura (it is a cavalry commander, so i move the charge attack to an hack attack bonus).

[fatherbushido]

remove another TODO noticed by elexis

[elexis]

• in lists.xml is an <item>attack_charge </item> It should be removed since there are also no actors that have that animation.
• some <History> tags contain "charge", but they can stay.
• the c++ part looks fine.
• "recharge" from cmpAttack.GetTimers() is unused apparently and should therefore be removed

Other than that there are no occurances of "Charge" or "Charging" (case insensitive search) in *cpp, *.h, *.js, *.json, *.xml, nor /art/actors/**

Can be committed if those 2 points are addressed, thanks for the patch!

[elexis]

In 18598:

Give the Carthaginian hero Maharbal a melee attack bonus instead of the not implemented charge bonus. Based on patch by fatherbushido, refs #4139.

[elexis]

In 18599:

Remove charge attack from the templates which isn't implemented nor designed yet. Patch by fatherbushido, fixes #4139.
Remove the unused "recharge" attack timer which was introduced in r7309 but should have been removed with r7775.