Remove DiscardInventory call in MushMatch.Killed

Fixes behaviour of Piñata and similar mutators, which use the inventory linked list on ScoreKill, which will be empty if DiscardInventory is called in the GameInfo's Killed overload function (which it shouldn't be, anyways - at least it isn't in DeathMatchPlus.Killed!).
wallabra · Feb 3, 2024 · e38b101 · e38b101
1 parent 332c56d
commit e38b101
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/Classes/MushMatch.uc b/Classes/MushMatch.uc
@@ -360,8 +360,6 @@ function Killed(Pawn Killer, Pawn Other, name DamageType)
     hMom = vect(0,0,0);
     dmg = 32767;
 
-    DiscardInventory(Other);
-
     if (Killer != None && Killer.bIsPlayer) {
         LastTauntTime   = Level.TimeSeconds + 6; // auto-taunts also make it easy to spot killers, disable momentarily
     }

diff --git a/buildconfig.sh b/buildconfig.sh
@@ -6,7 +6,7 @@
 export name="Mush Match"
 export package=MushMatch
 export version=1.3.5-pre3
-export build=20240601
+export build=20240602
 export debug=1
 export makeint=1
 export incl_readme=1

diff --git a/latest-changes.md b/latest-changes.md
@@ -3,3 +3,4 @@
 Implemented changes in Mush Match **v1.3.5** listed below.
 
 * Make special weapons (Sporifier, Beacon Launcher) only self-destruct if not owned.
+* Fix behaviour with Piñata-esque mutators that use the Inventory linked list on ScoreKill
Original file line number	Diff line number	Diff line change
Expand Up		@@ -3,3 +3,4 @@
		Implemented changes in Mush Match v1.3.5 listed below.

		* Make special weapons (Sporifier, Beacon Launcher) only self-destruct if not owned.
		* Fix behaviour with Piñata-esque mutators that use the Inventory linked list on ScoreKill