Thanks to Ukendio for the input regarding this issue!
The Route:query() method is misleading, the query keyword should be reserved for the ECS that YetAnotherNet is integrating with.
Suggested names for the method are:
Route:drain()Route:read()
Other names may be suggested in the comments here.
The QueryResult object should also be renamed to reflect this. These changes are also reserved for version 1.0.0 to avoid introducing breaking changes for issues like this, and instead fix them all in the official 1.0.0 release.
Thanks to Ukendio for the input regarding this issue!
The
Route:query()method is misleading, thequerykeyword should be reserved for the ECS that YetAnotherNet is integrating with.Suggested names for the method are:
Route:drain()Route:read()Other names may be suggested in the comments here.
The
QueryResultobject should also be renamed to reflect this. These changes are also reserved for version1.0.0to avoid introducing breaking changes for issues like this, and instead fix them all in the official1.0.0release.