Double strike works for ranged attackers too.
Squash a bug in AI attacking units w/ first-strike

The execute function took a non-const reference.  When the action
execution code modified the base action during a first-strike situation,
we inadvertently changed behavior.  Attackers would swing from more than
one hex away.
Pass Actions by value instead of making copies.