-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add QueryState::contains to mirror Query::contains #12750
Comments
Wouldn't the implementation for I'm not 100% on what the use case of adding |
Correct; this would simply be to give QueryState all the exact same tools as Query. |
…ate) (#12776) # Objective Fixes #12752. Fixes #12750. Document the runtime complexity of all of the `O(1)` operations on the individual APIs. ## Solution * Mirror `Query::contains` onto `QueryState::contains` * Make `QueryState::as_nop` pub(crate) * Make `NopWorldQuery` pub(crate) * Document all of the O(1) operations on Query and QueryState.
What problem does this solve or what need does it fill?
There exists
Query::contains
, but notQueryState::contains
.What solution would you like?
Add
QueryState::contains
.Additional context
Query::contains
was originally added in #3090 in 2021; QueryState wasn't generally accessible by users until #6083 in 2022, so just seems like an oversight.The text was updated successfully, but these errors were encountered: