Fix #258 and hopefully #210

CHANGELOG.md

@@ -30,6 +30,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 - In certain languages if translated chat messages with variable placeholders cannot be parsed at least still return the message rather than complete nonsense.
 - Fixed an issue where only a specific creep needs to be killed for AMAI to think mines and neutral buildings were unguarded going forwards.
 - Fixed an issue where town threat was always 0 or negative so AMAI couldn't tell it was in danger correctly.
+- Fixed an issue where AMAI would share heroes with each other on defeat which is not intended behaviour and could break score screens.
 - (Classic) Fixed game start crash with AMAI vs AI. (jzy-chitong56)
 - (Classic) Fixed some TFT settings not in sync with REFORGED.
 - (DevTools) Fixed the optimize all script and the makeAll and optimizeAll scripts will pause between versions.

Jobs/DETECT_DEFEAT.eai

@@ -109,7 +109,11 @@ function KillYourself takes boolean end_ai returns nothing
           loop
             set u = FirstOfGroup(g)
             exitwhen u == null
-            call SetUnitOwner(u, sharedRacePlayer, true)
+            if IsUnitType(u, UNIT_TYPE_HERO) then
+              call KillUnit(u)
+            else
+              call SetUnitOwner(u, sharedRacePlayer, true)
+            endif
             call GroupRemoveUnit(g, u )
           endloop
         endif