orion-ecs / keen-eye / 20145346487

65%

push

github

perf: Cache QueryDescription.AllRequired to eliminate LINQ allocations

Fixes #217

Changed AllRequired from a computed property using LINQ to a cached
ImmutableArray<Type> with lazy initialization. This eliminates multiple
enumerator allocations in hot paths during query archetype matching.

Implementation:
- Added nullable ImmutableArray<Type> cache field
- Lazy initialization on first access
- Cache invalidation when components are added (AddRead/AddWrite/AddWith)
- Return type changed from IEnumerable<Type> to ImmutableArray<Type>

Tests:
- Added test for cache instance reuse
- Added test for cache invalidation on component addition
- Added test to verify ImmutableArray return type

Co-authored-by: Tyler Coles <tyevco@users.noreply.github.com>

1210 of 1288 branches covered (93.94%)

Branch coverage included in aggregate %.

12 of 12 new or added lines in 1 file covered. (100.0%)

6631 of 7017 relevant lines covered (94.5%)

1.28 hits per line