Reduces the burst multiplier on the M39

[VileBeggar]

About the pull request

Reduces the burst scatter multiplier on the M39 by 43%, from 7 to 4.

Explain why it's good for the game

The weapon fails to shoot straight in burst mode even in its effective range. Automatic fire, which originally the M39 was touted as being good at when it was first being rolled out, is only slightly better than burst due to the fact the scatter ramps up. Neither modes are of any use in comparison to single fire, which is accurate at any range, including up to seven tiles. Even a M39 outfitted with the best scatter reducing attachments misses more shots than it has any right to.

In comparison, the MK2 rifle does not miss a single bullet in burst mode even at the edge of the screen, without a single attachment. A comparison video between the current firemodes and the tweaked ones are located in the testing section.

I personally think this is stupid and condemns the other two firemodes to never getting used. A 43% reduction from 7 to 4 is a lot, but this weapon could use some love beyond messing with damage or falloff (which I think are currently fine). With the current tweak, you still have to get really close to get use out of the firemodes if you have no attachments. However, applying a reflex sight or a vertical grip greatly enhances the use of the burst and auto mode, which is the way it should be:

As as side-note, this is the same tier the M39 had during its legendary BFG phase. However, since then we've lost the ability to attach the BFA to the M39, marines have completely lost access to the barrel charger and the M39 had its burst count lowered. Such a thing happening again shouldn't be a worry (hopefully).

Testing Photographs and Procedure

Screenshots & Videos

Old value:

m39old.mp4

New value:

m39new.mp4

Changelog

🆑
balance: M39 burst scatter multiplier reduced from 7 to 4.
/:cl:

[github-actions]

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

[Drulikar]

Code changes look fine

[VileBeggar]

i forgot to commit the right change. now it's 4 (instead of being 5).

[cuberound]

you change it from 4 to 7 but the changelog has it the other way

[VileBeggar]

you change it from 4 to 7 but the changelog has it the other way

that's just how the multipliers are defined in the code. TIER_7 = 4, TIER_4 = 7.
the bigger the tier, the smaller the number. don't ask me why

[Drulikar]

Seems fine