Fix Rocket/LTB/Caster ground firing and retire AMMO_EXPLOSIVE - #13054
Open
fira wants to merge 1 commit into
Open
Fix Rocket/LTB/Caster ground firing and retire AMMO_EXPLOSIVE#13054fira wants to merge 1 commit into
fira wants to merge 1 commit into
Conversation
Member
Author
|
Slapping Balance just for the hell of it because it turns out HPC has low scatter which is removed as a byproduct, that's really the only change i see though |
Contributor
|
Code LGTM |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
About the pull request
For about 4 years, Rocket Launchers and LTB have been unable to fire at the ground. Apparently nobody realized.
I actually fixed that bug for Flare Guns way back in 2022 in #2031 but in my grand ignorance I did not realize that AMMO_EXPLOSIVE implicitely means AMMO_HITS_TARGET_TURF because people were too lazy to type both.
Why does it break ?
As said in #2031 - essentially projectiles subjected to AMMO_HITS_TARGET_TURF must ignore scatter calculation. Scatter re-targets the projectile by raycasting it, so you're now aiming somewhere else than you clicked, far away. It forgets what turf you wanted to hit.
Won't removing scatter from LTB / M5 RPG / Caster be a big difference?
I don't think so, they're spec'ed to zero scatter anyway.
Technically the HPC and Dual Plasma Cannons have a small amount of scatter, but I don't think that's really the biggest worry if HPCs are in play.
Wait, you're removing AMMO_EXPLOSIVE entirely.
Yes because it actually only did two things:
The first obviously doesn't warrant a flag. The second... I don't know, I guess maybe if you were to somehow try to suicide point blank with a WP rocket it'd make a difference, but i'm pretty sure it's impossible too.
What about HE Mateba?
Well fixing without removing AMMO_EXPLOSIVE would give it ground targeting, which you don't want on small arms, so I didn't give it AMMO_HITS_TARGET_TURF. Happy ending.
HPC/RPG warning
This means HPC and RPG will explode where you click, even if you click 2 tiles from you. So be careful. Point at what you want to blow up, don't shoot in the general direction.
I can't believe you found a holy projectile flag in the land of terrible 24-flags BYOND drought!
Yeah well this goes to show how RELEVANT our flags are and why this is a false problem.
Changelog
🆑
fix: Rocket/LTB/HPC now explode on clicked turf properly again rather than flying by. Make SURE you click on what you want to explode, not the general direction.
del: Removed Explosive Ammo properties. It only mattered for BE/Suicide with HE Mateba, and it does that anyway.
balance: Caster Eradicator (HPC) / Dual Plasma Cannons had scatter disabled to target ground properly. Little difference that makes if that thing is pointed at you.
/:cl: