- Fixed a bug where texttags created by @Label could be orphaned due to a Blizzard bug which won't be fixed anytime soon.

Author: Motive

Commands/UnitCommands.galaxy

@@ -2201,6 +2201,8 @@ bool libcrtx_command_label(bool testConds, bool runActions)
 			TextTagAttachToUnit(tagId, UnitGroupLoopCurrent(), 1.0);
 
 			UnitSetCustomValue(UnitGroupLoopCurrent(), libcrtx_unit_custom_value_texttag, IntToFixed(tagId));
+			
+			libcrtx_labelmgr_addlabel(tagId, UnitGroupLoopCurrent());
 		}
 		UnitGroupLoopStep();
 	}

Common/Global.galaxy

@@ -73,6 +73,7 @@ string libcrtx_getspawnertype()
 include "Cortex/Common/Debug.galaxy"
 include "Cortex/Common/Settings.galaxy"
 include "Cortex/Common/Variables.galaxy"
+include "Cortex/Common/LabelManager.galaxy"
 include "Cortex/Common/Admin.galaxy"
 include "Cortex/Common/SharedData.galaxy"
 include "Cortex/Common/CortexID.galaxy"

Common/LabelManager.galaxy

@@ -0,0 +1,48 @@
+// Cortex SC2 Roleplaying Engine
+// Copyright (C) 2009-2011 <http://www.cortexrp.com/>
+//
+// This program is free software; you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation; version 2 of the License.
+// 
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+
+// This entire file's purpose would be useless if Blizzard
+// left us a viable way to detect when a unit died in a transport
+// ><
+// Bug report filed and ignored: http://us.battle.net/sc2/en/forum/topic/3811454102
+
+string libcrtx_labelmgr_prefix = "labelmgr_id_";
+int libcrtx_labelmgr_highId = -1;
+
+// Adds entry in dataTable for labelId -> unitHandle
+void libcrtx_labelmgr_addlabel(int labelId, unit unitHandle)
+{
+	if( labelId > libcrtx_labelmgr_highId ) {
+		libcrtx_labelmgr_highId = labelId;
+	}
+	
+	DataTableSetUnit(true, libcrtx_labelmgr_prefix + IntToString(labelId), unitHandle);
+}
+
+// Removes labels not tied to active units.
+void libcrtx_labelmgr_update()
+{
+	int i = -1;
+	unit linkedUnit;
+	
+	while( i <= libcrtx_labelmgr_highId )
+	{
+		linkedUnit = DataTableGetUnit(true, libcrtx_labelmgr_prefix + IntToString(i));
+		if( linkedUnit == null || !UnitIsAlive(linkedUnit) ) {
+			DataTableValueRemove(true, libcrtx_labelmgr_prefix + IntToString(i));
+			if( TextTagVisible(i, 1) ) {
+				TextTagDestroy(i);
+			}
+		}
+		i += 1;
+	}
+}

Common/Log.galaxy

@@ -60,7 +60,7 @@ static void libcrtx_log_event_global(int player, string message)
 	while(currentLevel < c_cortexMaxLoggedCommands)
 	{
 		libcrtx_log_globallog[currentLevel - 1] = libcrtx_log_globallog[currentLevel]; 
-		libcrtx_log_globallogplayers[c_currentLevel - 1] = libcrtx_log_globallogplayers[currentLevel];
+		libcrtx_log_globallogplayers[currentLevel - 1] = libcrtx_log_globallogplayers[currentLevel];
 		currentLevel += 1;
 	}
 	libcrtx_log_globallog[c_cortexMaxLoggedCommands - 1] = message;

Events/Events.galaxy

@@ -131,6 +131,7 @@ bool libcrtx_event_periodic_tick(bool checkConds, bool runActions)
 		i = i + 1;
 	}
 	libcrtx_ai_update();
+	libcrtx_labelmgr_update();
 	return true;
 }