Skip to content

Fix TF2_MakeBleed using incorrect custom damage type#1163

Merged
Headline merged 1 commit intoalliedmodders:masterfrom
FortyTwoFortyTwo:bleed-dmgcustom
Feb 9, 2020
Merged

Fix TF2_MakeBleed using incorrect custom damage type#1163
Headline merged 1 commit intoalliedmodders:masterfrom
FortyTwoFortyTwo:bleed-dmgcustom

Conversation

@FortyTwoFortyTwo
Copy link
Contributor

PR #965 accidentally changed TF2_MakeBleed custom damage type from 34 (TF_CUSTOM_BLEEDING) to 32 (TF_CUSTOM_SHOTGUN_REVENGE_CRIT).
I don't think using damage custom 32 change anything gameplay wise, but does incorrectly report 32 in player_death event, SDKHooks SDKHook_OnTakeDamage, and whatever other places damage custom uses.

Copy link
Member

@Headline Headline left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great catch, thank you!

@Headline Headline merged commit 1a71f4f into alliedmodders:master Feb 9, 2020
@FortyTwoFortyTwo FortyTwoFortyTwo deleted the bleed-dmgcustom branch February 9, 2020 09:55
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

Successfully merging this pull request may close these issues.

2 participants