[MIRROR] Assorted Ballistics Changes #1484
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Оригинальный PR: Baystation12#34297
🆑 Karl Johansson
rscadd: You can see what's currently in the chamber of a gun on examine.
rscadd: You can load non-magazine weapons by booping them with an appropriate loader.
tweak: Opening the bolt on bolt-action weapons unloads anything inside, not just empties.
tweak: 5mmR weapons are more accurate at long range.
/:cl:
"More information?"
When examining any gun that uses the
CYCLE_CASINGS
casing handling method (revolvers, flare guns, double-barreled shotguns, etc.) or a pump-action shotgun or a bolt action rifle with a weapons expertise skill of experienced or higher will allow you to view the chamber of the weapon so you can tell the status of the round inside, such as whether it's spent or not. This is because all of these weapons (although most egregiously revolvers) count empty cartridges in their round count which, while it does make sense and could even be considered realistic, can make using them harder. It also means that you can actually know what's going on inside your gun without having to code dive to find out how exactly it works.The next change is another quality of life change. It's been generalized a lot to allow for expansion later (still not certain about going all in on replacing shotholders with handfuls of rounds), but currently it serves to allow pump-action shotguns to be loaded by clicking on it with a shotgun shell holder in hand, rather than having to manually go back and forth between the two items. There's still a small cooldown when doing this so that you can't just spam shells into your shotgun.
The bolt action weapon changes are mostly to resolve a pet peeve of mine, although they do mean that now all guns are capable of ejecting loaded ammo, whereas before bolt action weapons were uniquely unable to do this. I also think it makes their operation more immersive and makes the decision of reloading them slightly more impactful.
The 5mmR cartridge has been changed to be as accurate as the 7mmR cartridge on the grounds that I felt that the nerf was somewhat arbitrary and I wasn't satisfied with the explanation I was given as to why they were nerfed, since there are 3 different sniper weapons that use the calibre.
Spellcheck's a spellcheck. "it's" is a contraction of "it is". "its" is the possessive form of "it".
"Is there bugs?"
I hope not. I've tested this to the best of my abilities and I believe I've resolved everything. Unfortunately I am also me, so there might still be oversights. On the bright side, I never saw any runtimes. If this merge conflicts with something else, I'll probably just remove the part that is conflicting and try and add it again later, if that's allowed.
"My eyes are bleeding."
I've done my best to lay things out in a roughly coherent manner, although I'm not particularly well-versed in good coding practices. I'm completely open to redoing this from the ground up if it's required.
"I don't agree with these changes."
I will remove changes that are deemed unnecessary or break contribution guidelines if it is requested.