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

Bug: Rocket Launcher stays centered when switching from another weapon #1926

Open
Wavy667 opened this issue Sep 27, 2024 · 3 comments
Open

Comments

@Wavy667
Copy link

Wavy667 commented Sep 27, 2024

When Weapon Alignment is set to Centered, the Rocker Launcher sprite on the HUD remains centered when switching from another firing weapon despite the Rocket Launcher not firing itself.

Reproducing

  • Set Weapon Alignment is set to Centered
  • Hold down fire on any weapon that isn't the Rocket Launcher
  • While fire is held down, switch to the Rocket Launcher
  • The Rocket Launcher sprite remains centered on the HUD
@fabiangreffrath
Copy link
Owner

Same in DSDA-Doom. This is because the Rocket Launcher (and the BFG, btw) aren't actually firing when switched to in attacking mode. I don't see much of a problem in this, though.

@Wavy667
Copy link
Author

Wavy667 commented Sep 27, 2024

I know the firing lock is intended behaviour, but visually it doesn't match up compared to when Weapon Alignment is set to default because when switching to the Rocket Launcher or BFG, they still sway unlike when Weapon Alignment is set to Centered (which is more natural IMO since they are not actually firing).

@fabiangreffrath
Copy link
Owner

I think the reason is that we are checking for player->attackdown here instead of player->cmd.buttons & BT_ATTACK and I think there was a reason for this, but I can't remember right now.

woof/src/p_pspr.c

Line 1160 in a55429f

else if (!player->attackdown || center_weapon_strict == WEAPON_BOBBING)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants