MantisBT - Zandronum
View Issue Details
0004160Zandronum[All Projects] Bugpublic2023-08-31 18:102024-03-23 20:58
Kaapeli47 
Kaminsky 
normalminoralways
resolvedfixed 
MicrosoftWindowsXP/Vista/7
3.1 
3.23.2 
0004160: Skin death sound does not play if the player respawns before death sound
Player skin death sound does not play if the player leaves the corpse by respawning before A_PlayerDeath or A_XDeath happens. Instead, the default death sound plays, such as Doomguy's death scream in Doom.

This is most noticeable in classic Doom deathmatch servers with a player skin pack and Compat_InstantRespawn set to true.
Load a skin that uses one or more death sounds. Start a classic Doom deathmatch game with Compat_InstantRespawn set to true. Get fragged and try to respawn as soon as you can, at least before the death sound plays on your corpse. It should play Doomguy's death scream instead of the skin's.
Issue found in Euroboros classic Doom 2 deathmatch servers, such as [EB] Greenwar Deathmatch and [EB] Exec & Brit Deathmatch, which use IDMSkinpack_v1.36.pk3 as an optional file.

Temporary solution/workaround: Set Compat_InstantRespawn to false, and set SV_RespawnDelayTime to 0.3 seconds. This is enough time for the death sound to execute before the player respawns, so the skin's death sound actually plays. However, while it is hardly noticeable, the respawning isn't in fact instant anymore. This setting is now in use in the aforementioned Euroboros classic deathmatch servers after this issue was found.
No tags attached.
Issue History
2023-08-31 18:10Kaapeli47New Issue
2024-03-17 17:59KaminskyNote Added: 0023418
2024-03-17 17:59KaminskyAssigned To => Kaminsky
2024-03-17 17:59KaminskyStatusnew => needs review
2024-03-17 17:59KaminskyProduct Version4.0-beta => 3.1
2024-03-17 17:59KaminskyTarget Version => 3.2
2024-03-17 21:47KaminskyNote Added: 0023419
2024-03-17 21:47KaminskyStatusneeds review => needs testing
2024-03-23 20:58Ru5tK1ngNote Added: 0023475
2024-03-23 20:58Ru5tK1ngStatusneeds testing => resolved
2024-03-23 20:58Ru5tK1ngResolutionopen => fixed
2024-03-23 20:58Ru5tK1ngFixed in Version => 3.2

Notes
(0023418)
Kaminsky   
2024-03-17 17:59   
I created a new merge request to address this issue:'https://foss.heptapod.net/zandronum/zandronum-stable/-/merge_requests/68 [^]'
(0023419)
Kaminsky   
2024-03-17 21:47   
This got pushed in:'https://foss.heptapod.net/zandronum/zandronum-stable/-/commit/75a8bf29c784ec8466d594767b0d72a6d053135d [^]'
(0023475)
Ru5tK1ng   
2024-03-23 20:58   
Kaapeli tested out build 240317 that included this fix and reported the issue was resolved.