2 replies to this topic

[LarkinVB]

I'd like to see some code cleanup.

Remove all useless variables like attack_rating, stop_attack_rating etc.
Simplify variables which do use the scale.
We have MinArmyStrengthPerTier but also min_units which basically transfers to min_units * 250. Therefore min_units should be changed to min_army.

When I did the transition to 1.75 I wasted some time tweaking variables which aren't used at all.

You get the idea.

[LarkinVB]

More code cleanup :

Please outsource the force tech time from core to loader.ai GetInfo(sType)

Please use the outsourced "TurretPower" value for the various calculations where the core code still adds fixed +300 to army for buildings with weapons.

[ArkhanTheBlack]

Okay! Will change that for next beta.