CleverRaven/Cataclysm-DDA — pull request #1666 by atomicdryad (2013-06-20 12:37:44+00:00)
Title: Ranged burst now fires more than 1 aimed shot, retargets properly, and is more readable and comented.

Previously, all burst fire shot once at a target, and:
* Promptly forgot if the target moved, or on misses.
* Picked targets closest to shooter, not closest to last position.
* If no targets within a range of range of 2 + gun skill, it would fire at the original position, remaining somewhat effective on sessile targets or 'it's coming right at us' scenarios.

Now it:

* Iterates from last target position, up to 2+gun skill (spiraling outward / walking the burst to nearby targets)
* Only accepts non-friendly targets within firing range
* Rejects targets at shooter coordinates
* Code further down (meant to add tracer effect on misses) that randomized the current target position variables now uses temporary variables.

In addition:
* The re-targeting section has been cleaned up, and commented enough so hopefully 'radius' isn't mistaken for 'range' again.
* (disabled) debug output (which should help diagnose inevitable future issues)