Projectile overrange unit implementation

[SethDGamre]

Adds overrange to StarburstLauncher/MissileLauncher units to prevent overrange. It's +10% of unitdef.range. This PR includes legion units. AA units are excluded.

The legion unitdefs are pretty terribly un-alphabetized so there's alot of changes being that I used the unit bake method of adding these customparams to units

[saurtron]

Shouldn't have reorganized alphabetically as part of the same PR. That makes the pr more difficult to review, and also pollutes git history.

Ok I see it's because of some tool, but git allows adding just some of the stuff (git add -p). Anyways not a big deal.

[saurtron]

this file doesn't look like it should be part of the commit, right?

[saurtron]

unit bake method

looking at the changes imo that method should be forbidden XD, would have been better to just edit manually or even code postprocess.

[sprunk]

proper capitalisation, post-processing?

See https://github.com/beyond-all-reason/spring/blob/master/doc/site/articles/unit-defs.markdown#unit-def-files

baking should be forbidden

Yes.

[saurtron]

proper capitalisation, post-processing?

See https://github.com/beyond-all-reason/spring/blob/master/doc/site/articles/unit-defs.markdown#unit-def-files

That examples remakes all keys to lowercase, but not sure bar is doing that (guess it is tho). Doesn't seem like a very good idea, now everyone changing capitalization randomly just in case.

[sprunk]

Read the text of the article and not just the example. tl;dr:

• engine doesn't care about capitalisation.
• making sure everything is lowercase is typically either done by just running lowerkeys in postprocessing (which is the implementation in the article's example), or by keeping all individual defs lowercase (which BAR does). There are tradeoffs but ultimately it doesn't matter which one.
• the reason to do this is that Lua post-processing cares. Most of the Recoil ecosystem has postprocessing that expects everything it uses to be completely lowercase, including BAR (e.g. https://github.com/beyond-all-reason/Beyond-All-Reason/blob/master/gamedata/unitdefs_post.lua#L177-L181 )

[drivver44]

Do not use Unitdef baking.
its a great way to break any of our open PRs that deal with unitdefs and causes a whole bunch of merge conflicts that have to be resolved.