collector's bug

[1115803755]

Hi,

I have found a bug want to contribute.

Here's some code

public void Activate() { for (int i = 0; i < _groups.Length; i++) { var group = _groups[i]; var groupEvent = _groupEvents[i]; switch (groupEvent) { case GroupEvent.Added: group.OnEntityAdded -= _addEntityCache; group.OnEntityAdded += _addEntityCache; break; case GroupEvent.Removed: group.OnEntityRemoved -= _addEntityCache; group.OnEntityRemoved += _addEntityCache; break; case GroupEvent.AddedOrRemoved: group.OnEntityAdded -= _addEntityCache; group.OnEntityAdded += _addEntityCache; group.OnEntityRemoved -= _addEntityCache; group.OnEntityRemoved += _addEntityCache; break; } } }

As you can see, "group.OnEntityRemoved += _addEntityCache;" is add wrong EventHandle

[c0ffeeartc]

Collector caches entities on Added/Removed/AddedOrRemoved, it looks as intended.

[1115803755]

I mean why did group.OnEntityRemoved and group.OnEntityAdded add the same EventCache

[c0ffeeartc]

I mean why did group.OnEntityRemoved and group.OnEntityAdded add the same EventCache

group.OnEntityRemoved and group.OnEntityAdded can add to the same HashSet. Based on how Collector was set up.

[1115803755]

Think you, I got it wrong.