Manager: Fix performance issue when searching for commander tags

By not saving the found events, this ended up going through all the events for each player.

ArcdpsLogManager/CHANGELOG.md

@@ -3,9 +3,14 @@
 This is the full changelog of the arcdps Log Manager.
 
 ## Log Manager v1.10
+
 #### New features
 - Added a button to set the dps.report user token in the settings
 - Added max parallel threads setting to the Game data gathering tab (View -> Debug Data must be enabled to see this tab).
+
+#### Performance improvements
+- Improved log processing performance by ~25% for 10man logs, even more in WvW logs.
+
 #### Fixes
 - Fixed log detail pane not appearing for selected logs processed with Log Manager versions 1.3 and older.
 - Fixed a stray debug data section appearing when multiple logs were selected even if debug data wasn't enabled

ArcdpsLogManager/Logs/LogData.cs

@@ -276,7 +276,7 @@ public void ProcessLog(LogAnalytics logAnalytics)
 					HealthPercentage = 0;
 				}
 
-				var tagEvents = log.Events.OfType<AgentTagEvent>().Where(x => x.Marker.Id != 0 && x.Agent is Player);
+				var tagEvents = log.Events.OfType<AgentTagEvent>().Where(x => x.Marker.Id != 0 && x.Agent is Player).ToList();
 				Players = analyzer.GetPlayers().Where(x => x.Identified).Select(p =>
 					new LogPlayer(p.Name, p.AccountName, p.Subgroup, p.Profession, p.EliteSpecialization,
 						GetGuildGuid(p.GuildGuid))