Skip to content

Use memmem when searching for usages in ide-db #13243

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

Merged
merged 1 commit into from
Sep 16, 2022

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Sep 16, 2022

We already have this dependency, so there is no reason not to use it as it is generally faster than std in our use case.

@Veykril
Copy link
Member Author

Veykril commented Sep 16, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Sep 16, 2022

📌 Commit b73fa0b has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Sep 16, 2022

⌛ Testing commit b73fa0b with merge b6e3f41...

@bors
Copy link
Contributor

bors commented Sep 16, 2022

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing b6e3f41 to master...

@bors bors merged commit b6e3f41 into rust-lang:master Sep 16, 2022
@Veykril Veykril deleted the search-memchr branch September 16, 2022 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants