Skip to content

Various defuser fixes and code refactory#848

Merged
s1lentq merged 5 commits intorehlds:masterfrom
dystopm:defuser-code-clean
Sep 5, 2023
Merged

Various defuser fixes and code refactory#848
s1lentq merged 5 commits intorehlds:masterfrom
dystopm:defuser-code-clean

Conversation

@dystopm
Copy link
Contributor

@dystopm dystopm commented Jul 14, 2023

  • Fixed item_thighpack entity not spawning on CBasePlayer::Disappear (which shares logic with CBasePlayer::Killed). Fixed version deleted GiveNamedItem (reasonably) but nobody coded the entity spawning.
  • Fixed CCSPlayer::RemovePlayerItemEx not clearing ProgressBar HUD when removing defuser.
  • Ensuring m_bIsDefusing assignation whenever defuser is removed from player.
  • Defuser removal logic grouped into CBasePlayer::RemoveDefuser.
  • Some minor logic changes and unneeded code erase.

@dystopm
Copy link
Contributor Author

dystopm commented Jul 14, 2023

Just found RemoveDefuser is a function not present in original version so it gaves me room to not enclose some code in fix macros. Gonna check now

@dystopm dystopm marked this pull request as draft July 14, 2023 06:34
@dystopm dystopm changed the title Fix: Defuser not spawning on CBasePlayer::Disappear (plus code cleaning) Various defuser fixes and code refactory Jul 15, 2023
@dystopm dystopm marked this pull request as ready for review July 15, 2023 06:06
@s1lentq s1lentq merged commit facc2be into rehlds:master Sep 5, 2023
@dystopm dystopm deleted the defuser-code-clean branch October 11, 2023 15:10
yohimik pushed a commit to yohimik/ReGameDLL_CS that referenced this pull request Jun 5, 2025
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