Fixes for EntityIterator and IndicesIterator

louisponet / Overseer.jl

Entity Component System with julia

Other

59 stars 4 forks source link

Fixes for EntityIterator and IndicesIterator #5

Closed c42f closed 3 years ago

c42f commented 3 years ago

IteratorSize(::IndicesIterator) set to SizeUnknown, as it involves a predicate which filters out an unknown number of elements.
IteratorSize(::EntityIterator) defers to the wrapped iterator
Implement eltype(::EntityIterator) so that collect() is type stable

Fixes #4

louisponet commented 3 years ago

Thank you! I didn't know about Base.IteratorSize and SizeUnknown. Indeed you are right, assigning an actual iterator length to iterators with a test function is not correct.