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 support for multiple burst modes #5609

Closed
NaturesWitness opened this issue Jan 19, 2014 · 4 comments

Comments

Projects
None yet
4 participants
@NaturesWitness
Copy link
Contributor

commented Jan 19, 2014

Was reading some comments on the forums, and it seems a lot of people would like a way to fire smaller bursts with some of the full-auto weapons. It occurred to me, would it be possible to change the "burst" field in the item jsons to accept multiple values?

For example, a weapon could have "burst": [3, 10] and be able to cycle from single shot, 3-round burst, and 5-round burst with the burst mode key.

How hard would it be to implement something like this?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@kevingranade

This comment has been minimized.

Copy link
Member

commented Jan 19, 2014

Not terribly hard, some minor surgery on game::fire(). I've been meaning
to do this for... over a year now. (I added the initial firing mode last
christmas)

@NaturesWitness

This comment has been minimized.

Copy link
Contributor Author

commented Jan 19, 2014

It's good to hear this should be durable. I was considering trying to make a select-fire gun using the Heatblade weapon switching code, which would have been a huge, cluttered mess. I'll just wait until someone has a chance to implement the game::fire method instead.

@chaosvolt

This comment has been minimized.

Copy link
Contributor

commented Jun 9, 2015

This would also mesh well with guns that are fully-automatic but lack selective fire, if you can assign randomly-selected burst lengths from a range of minimum and maximum shots.

@mugling

This comment has been minimized.

Copy link
Contributor

commented Jul 2, 2016

Now supported in master

@mugling mugling closed this Jul 2, 2016

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.