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

Dropping Infinite Pistol clips when Reloading #570

Open
PixelPone897 opened this issue Dec 11, 2020 · 6 comments
Open

Dropping Infinite Pistol clips when Reloading #570

PixelPone897 opened this issue Dec 11, 2020 · 6 comments
Labels
minor priority Not a big issue/problem

Comments

@PixelPone897
Copy link

Hello again.
It is possible to drop infinite clips when reloading the UAC-B950 .45 pistol.

Method to perform this bug (it's very easy to do):

  1. Empty clip of pistol.
  2. Reload.
  3. Switch to another weapon before the reload animation is complete.
  4. A pistol clip should spawn on the ground even though you did not fully reload the pistol.

PB Infinite Pistol Clip Drop Bug

This was tested on the most current master branch version of Project Brutality (last commit = 11/30/20) on GZDoom 4.5,
E1M1 of Ultimate Doom.

@JMartinezHarmoniousPerson
Copy link
Contributor

It's due to the A_WeaponReady functions that allow this, along with the A_CustomMissile being called when reloading a pistol. I'm actually planning on making an edit Reload state that a magazine is not spawned when pistol ammo is below zero. This may not be fixable due to script limitations, and is a low priority.

@PixelPone897
Copy link
Author

PixelPone897 commented Dec 11, 2020

Should I assign it to low priority? Or just leave it? (I can't label the issue properly)

@JMartinezHarmoniousPerson
Copy link
Contributor

Notes can only be assigned by a collaborator. Ask a collaborator to assign it.

@PixelPone897
Copy link
Author

Alright

@PixelPone897
Copy link
Author

@goldenmob72 When you have time, it is possible for you to add the minor priority tag to this issue? Thanks.

@D3X1KAXYZ D3X1KAXYZ added the minor priority Not a big issue/problem label Dec 11, 2020
@D3X1KAXYZ
Copy link
Collaborator

Tag applied

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor priority Not a big issue/problem
Projects
None yet
Development

No branches or pull requests

4 participants