Skip to content

Clarify that match_indices returns an iterator over byte indices - #161825

Open
zepredos wants to merge 1 commit into
rust-lang:mainfrom
zepredos:doc-match-indices-fix
Open

Clarify that match_indices returns an iterator over byte indices#161825
zepredos wants to merge 1 commit into
rust-lang:mainfrom
zepredos:doc-match-indices-fix

Conversation

@zepredos

Copy link
Copy Markdown
Contributor

Addresses #161812
Ran the following tests:
./x doc library/core
./x test library/core
Successfully built docs website via
./x doc library/core --open

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 26, 2026
@rustbot rustbot added the T-libs Relevant to the library team, which will review and decide on the PR/issue. label Aug 26, 2026
@rustbot

rustbot commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the pull request, and welcome! The Rust Project has assigned @JohnTitor (or someone else) to review your changes, you should hear from them (or someone else) within the next two weeks.

Please see the contribution instructions and our LLM policy for more information.

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: libs
  • libs expanded to 12 candidates
  • Random selection from JohnTitor, Mark-Simulacrum, clarfonthey, nia-e

@zepredos
zepredos force-pushed the doc-match-indices-fix branch from 24ebcb6 to f70261b Compare August 26, 2026 15:59
@rustbot

rustbot commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@zepredos

Copy link
Copy Markdown
Contributor Author

@rustbot label +A-docs

@rustbot rustbot added the A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools label Aug 26, 2026
@JohnTitor

Copy link
Copy Markdown
Member

Could you also tweak the wording for rmatch_indices? It should have the same wording.

@JohnTitor JohnTitor added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-docs Area: Documentation for any part of the project, including the compiler, standard library, and tools S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants