Pop_Slime/crtr
1
0
Fork 0
Code Issues 6 Pull Requests Projects 1 Releases Activity

Fix effect queue with multiple items of the same instance.

Browse Source
This commit is contained in:
PopSlime
2023-02-19 22:15:25 +08:00
parent 3728360dd2
commit f2f3dba098
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
Assets/Cryville/Crtr/EffectGroup.cs
View File

@@ -43,14 +43,17 @@ namespace Cryville.Crtr {
} }
public void Emit(float index) { public void Emit(float index) {
EffectInstance instance; EffectInstance instance;
if (!_instances.TryGetValue(index, out instance)) { if (_instances.TryGetValue(index, out instance)) {
var i = _endQueue.BinarySearch(instance);
_endQueue.RemoveAt(i);
}
else {
_instances.Add(index, instance = _pool.Rent()); _instances.Add(index, instance = _pool.Rent());
} }
instance.Index = index; instance.Index = index;
instance.OnEmit(_time); instance.OnEmit(_time);
var i = _endQueue.BinarySearch(instance); var i2 = ~_endQueue.BinarySearch(instance);
if (i < 0) i = ~i; _endQueue.Insert(i2, instance);
_endQueue.Insert(i, instance);
} }
public void Dispose() { public void Dispose() {
_pool.DisposeAll(); _pool.DisposeAll();