feat(balance): Removing damage Penalty for the Suppressors

[spyfox321]

Removed Damage Penalty from the makeshift suppressors and tweaked the handling of the MP5SD integral suppressor.

The damage modifier for the MP5SD suppressor has Intentionally not changed to reflect its unique subsonic internal suppressor.

Purpose of change

Suppressors in real life don't decrease the Velocity of bullets, decreasing the damage. The normal suppressors don't have a negative damage modifier, so I removed them to make consistent.

Describe the solution

Fully removed the damage penalty from makeshift suppressor, soda bottle suppressor, and shotgun suppressor.

I've intentionally left the MP5SD's internal suppressor as it's a uniquely designed suppressor to slow the bullet to a subsonic round.

Describe alternatives you've considered

Theoretically, if the loudness values are great enough, I think as a balance mechanic it would be viable to keep the damage penalty. But from my experience, I considered the loudness modifiers as "realistic" so I wanted to tweak the modifiers as such.

I would've adjusted the suppressor's loudness modifier too to make the suppressors most realistic, but just by checking the JSON, it's hard to check how loud the are guns and what the loudness values truly mean.

Testing

Additional context

Currently, it's very unclear to me how much of what is loudness value means. if someone can/want to adjust the loudness modifiers too, feel free.

Checklist

[autofix-ci]

Autofix has formatted code style violation in this PR.

I edit commits locally (e.g: git, github desktop) and want to keep autofix

1. Run git pull. this will merge the automated commit into your local copy of the PR branch.
2. Continue working.

I do not want the automated commit

1. Format your code locally, then commit it.
2. Run git push --force to force push your branch. This will overwrite the automated commit on remote with your local one.
3. Continue working.

If you don't do this, your following commits will be based on the old commit, and cause MERGE CONFLICT.

[scarf005]

feat(balance): Changing Values for the Suppressors

could we have a more descriptive PR title? from the PR title alone it's not obvious what the changes are (e.g is it a nerf? is it a buff?)

[spyfox321]

I would actually buff the MP5SD's suppressor, but I really can't tell how much is -32 loudness modifier, so I'll leave it to people who know what these values mean more.

[VissValdyr]

I would actually buff the MP5SD's suppressor, but I really can't tell how much is -32 loudness modifier, so I'll leave it to people who know what these values mean more.

The easiest way is to test them out. One step is 5 sound, if I recall, so build it from there. You can debug spawn a friendly NPC and give him a mp5sd, spawn a enemy and save. Let him shoot him and then consider, how far away do you have to be to hear his firing as loud as footsteps. If it is too far in game, make the number higher than -32, for example -40. Save the value and test again until you are happy with the findings. You can easily test jsons by going to the main menu (without saving via debug) and load the save with the new values again.