Toreole / BasicCombatlogParser

A small project with limited functionality in parsing WoW Combatlogs.
0 stars 1 forks source link

Remove Sparse Arrays if possible #19

Closed Toreole closed 11 months ago

Toreole commented 1 year ago

Already did this with strings where i went from 8MB to 1MB of duplicate strings But there are still 4MB of sparse arrays. some offenders that are guaranteed to be because of me:

Toreole commented 1 year ago

Okay heres a funny thing at the end of ParseEncounter, there is only one combatlogevent[] array. but after the method returns, and e.g. you get to GenerateDamageDoneBreakdown in the SingleEncounterView, there are suddenly 3 CombatlogEvent[] arrays:

Toreole commented 1 year ago

Update: the empty array was being created by the default value of the EncounterInfo.CombatlogEvents property. that is gone now.

Toreole commented 1 year ago

grafik i really do not understand why empty arrays, that arent referenced by anything, are not freed up?