Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add speedloaders for many guns #29366

Merged
merged 4 commits into from Apr 9, 2019

Conversation

Projects
None yet
6 participants
@Rail-Runner
Copy link
Contributor

commented Apr 7, 2019

Summary

SUMMARY: Content "Add speedloaders for many guns, mainly revolvers"

Purpose of change

Adds speedloaders for revolvers, for a much faster reloading option for many guns. Reloading without them is still possible.

Describe the solution

Add speedloader items, their spawns, and set guns to use them via JSON. Also adds a recipe for tube loader for Marlin 39A.

Describe alternatives you've considered

  1. Adding crafting recipes for speedloaders. Couldn't find any info on how they could be improvised, though.
  2. Set player-crafted revolvers to use speedloaders too. No idea what type of revolver they are, though; depending on design, speedloaders might not work.

Additional context

Weight and price of speedloaders are wild guesses; would be good if someone could provide actual info on that.

Rail-Runner added some commits Apr 7, 2019

@Throwaway-name

This comment has been minimized.

Copy link

commented Apr 8, 2019

Did you test these changes with extended realistic guns mod active?

"DEBUG: Warnings for type sw_610:
specified magazine but none provided for default ammo type"

You need to add a 10mm version to extended realistic guns.

@Rivet-the-Zombie

This comment has been minimized.

Copy link
Member

commented Apr 8, 2019

Keep in mind that some revolvers have a loading gate that prevents the usage of speedloaders.

@kevingranade

This comment has been minimized.

Copy link
Member

commented Apr 8, 2019

Echoing @Rivet-the-Zombie 's comment, please do some basic research and determine if a given revolver is at least capable of accepting a speedloader before adding one.

Having said that, I spot-checked all the non-fictional guns you've added speedloaders to, and they all seem to have swing-out cylinders, so they should be fine.

I'd consider making some of the very largest caliber fictional guns fixed-frame, but I won't insist.

@Rail-Runner

This comment has been minimized.

Copy link
Contributor Author

commented Apr 8, 2019

Did a bit of research on speedloaders.

Ruger LCR .22 speedloader: https://www.youtube.com/watch?v=8y8JiNhTkfc
Ruger LCR .38 speeloader: https://www.youtube.com/watch?v=e-9Y_Pv7_RE
S&W 610 speedloader: https://www.youtube.com/watch?v=6724UKl2lz0 (it shows moon clips, but this probably means that a speedloader is possible too)
S&W 629 speedloader: https://www.youtube.com/watch?v=wGeVwjLpzF0 (doesn't seem to note that the gun shown is exactly S&W 629, but it is a .44 Magnum S&W revolver, so it should be good enough)
Ruger Redhawk speedloader: https://www.youtube.com/watch?v=FRiV3Pqc3_g (no idea if it and S&W 629 use the same speedloader, though, but amount of rounds that fit inside and ammunition used seem to be the same)
Taurus Raging Bull speedloader: https://www.amazon.com/5-Star-Firearms-Taurus-Loader/dp/B007JL4IVA (one comment seems to say that it works for Taurus Raging Bull .454 Casull specifically)
S&W 500 speedloader: https://www.youtube.com/watch?v=DLybRf4n-Ao (doesn't actually show loading, but video name assumes that it does work for S&W 500)
Marlin 39A speedloader: https://www.youtube.com/watch?v=fJubPAFve2s (even shows that it is possible to make)

@Throwaway-name

This comment has been minimized.

Copy link

commented Apr 8, 2019

You also need to add this to the sw_610 in realguns/gun.json to fix the bug:
"magazines": [ [ "10mm_auto", [ "40_speedloader6" ] ] ],

Sorry, I shouldn't have posted the test ID instead of the PR ID.

@Rail-Runner

This comment has been minimized.

Copy link
Contributor Author

commented Apr 9, 2019

Done, should be working now.

@kevingranade kevingranade merged commit eaf3290 into CleverRaven:master Apr 9, 2019

2 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
gorgon-ghprb Build finished.
Details

@Rail-Runner Rail-Runner deleted the Rail-Runner:speedloader branch Apr 9, 2019

@jimmydoh

This comment has been minimized.

Copy link
Contributor

commented Apr 9, 2019

Still breaks with Extended Realistic Guns mod active.

Typo in realguns/magazine.json:

"copy_from" instead of "copy-from" and needs a "name" or throws member not found: name and can't load a save / start a new game.

@kevingranade

This comment has been minimized.

Copy link
Member

commented Apr 10, 2019

This pull request has been mentioned on Cataclysm: Dark Days Ahead. There might be relevant details there:

https://discourse.cataclysmdda.org/t/latest-experimental-features/5582/1106

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.