-
-
Notifications
You must be signed in to change notification settings - Fork 4k
Closed
Labels
A-ECSEntities, components, systems, and eventsEntities, components, systems, and eventsC-Code-QualityA section of code that is hard to understand or changeA section of code that is hard to understand or changeC-PerformanceA change motivated by improving speed, memory usage or compile timesA change motivated by improving speed, memory usage or compile times
Description
What problem does this solve or what need does it fill?
As discussed in #3765, these APIs no longer appear to be measurably faster than query.iter()
mechanisms after #2254.
Having two ways to do the same thing is not ideal, and this API is generally less flexible and considered harder to read.
What solution would you like?
Remove these APIs.
Additional context
Some ballpark benchmarks were done in #3765, but this change should be accompanied by higher quality testing.
Metadata
Metadata
Assignees
Labels
A-ECSEntities, components, systems, and eventsEntities, components, systems, and eventsC-Code-QualityA section of code that is hard to understand or changeA section of code that is hard to understand or changeC-PerformanceA change motivated by improving speed, memory usage or compile timesA change motivated by improving speed, memory usage or compile times