Removed duplication of attack logging text after performing fire turndown
Fixed indentation of comment
Fixed action logging when there are either multiple attackers or multiple defenders
Used implode() to join multiple attacker or defender log elements
Fixed checkstyle warning
Added no-op change to force Jenkins to rerun
Merge remote-tracking branch 'upstream/master' into 1128_fire_log
Refactored action logging for fire to cache fire info until BMGame->commit_attack() and then build the log entry there
Removed duplication of initial attack choice
Refactored to satisfy PMD
Merge pull request #1142 from blackshadowshade/1128_fire_log

Removed duplication of attack logging text after performing fire turndown