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

Fix 40x53mm copy-from issues #42292

Merged
merged 2 commits into from Jul 20, 2020
Merged

Fix 40x53mm copy-from issues #42292

merged 2 commits into from Jul 20, 2020

Conversation

ghost
Copy link

@ghost ghost commented Jul 20, 2020

Summary

SUMMARY: None

Purpose of change

The volume field for 40x53 grenades was not put on the abstract round, leading to non-M1001 grenades being microscopically tiny. The casing and weight can also be moved to the abstract, since all of them use the M169 casing & weigh 340g.

Describe the solution

Move casing, volume, & weight to the abstract grenade.

Describe alternatives you've considered

None.

Testing

Spawn in every ammo type to view its volume, fire off a Mark 19 to see that casing is inherited from copy-from.

Additional context

Original bug(from devcord):
image

@ZhilkinSerg ZhilkinSerg added <Bugfix> This is a fix for a bug (or closes open issue) Items: Ammo / Guns Ammunition for all kinds of weapons and these weapons themselves [JSON] Changes (can be) made in JSON labels Jul 20, 2020
@ZhilkinSerg ZhilkinSerg merged commit 08113fe into CleverRaven:master Jul 20, 2020
@ghost ghost deleted the GrenadeVolFix branch July 20, 2020 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) Items: Ammo / Guns Ammunition for all kinds of weapons and these weapons themselves [JSON] Changes (can be) made in JSON
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants