Skip to content

refactor: remove ad-hoc subslice_range - #17390

Merged
epage merged 1 commit into
rust-lang:masterfrom
weihanglo:198
Aug 25, 2026
Merged

refactor: remove ad-hoc subslice_range#17390
epage merged 1 commit into
rust-lang:masterfrom
weihanglo:198

Conversation

@weihanglo

Copy link
Copy Markdown
Member

slice::subslice_range API is stabilized in 1.98

🤖 LLM disclosure: I asked LLM to show if there is any cleanup we can do with 1.98. It pointed out some. Code was hand-written and never generated.

The rest were not interesting at all. One in frontmatter.rs for substr_range, though I guess we want to keep it as close as rust-lang/rust's impl (or it has been diverged I forgot). Others were str::strip_circumfix and Path::is_empty.

This API is stabilized in 1.98
@rustbot rustbot added A-registries Area: registries S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 25, 2026
@rustbot

rustbot commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

r? @epage

rustbot has assigned @epage.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @epage, @weihanglo
  • @epage, @weihanglo expanded to epage, weihanglo

@epage
epage enabled auto-merge August 25, 2026 01:34
@epage

epage commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

frontmatter.rs is an independent impl and we are free to refactor. We do need to update r-a's fork at some point.

@weihanglo

Copy link
Copy Markdown
Member Author

frontmatter.rs is an independent impl and we are free to refactor. We do need to update r-a's fork at some point.

I wonder if that deserves a new cargo-util-frontmatter package.

@epage

epage commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Maybe, eventually. Part of the move to utils was prep for that one day. For r-a, I'm unsure if it would help because they likely wouldn't want to wait for a release.

@epage
epage added this pull request to the merge queue Aug 25, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 25, 2026
@epage
epage added this pull request to the merge queue Aug 25, 2026
Merged via the queue into rust-lang:master with commit 261e135 Aug 25, 2026
29 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 25, 2026
rust-bors Bot pushed a commit to rust-lang/rust that referenced this pull request Aug 26, 2026
Update cargo submodule

7 commits in e8cb624d5701824f46a2ec5873cfd59ee3d2f66c..94ba974179df2adb3c911fadf361f03b84aa8f14
2026-08-22 00:23:45 +0000 to 2026-08-25 18:16:11 +0000
- feat(diag): Stabilize cargo-lints  (rust-lang/cargo#17298)
- chore(deps): Update partial_ref to v0.3.4 (rust-lang/cargo#17392)
- refactor: remove ad-hoc `subslice_range` (rust-lang/cargo#17390)
- docs(changelog): move build-dir new layout to Changed (rust-lang/cargo#17387)
- chore(deps): update msrv (1 version) to v1.98 (rust-lang/cargo#17386)
- docs: Use mdbook admonitions (rust-lang/cargo#17384)
- chore(ci): exclude resolver-tests from intra doc link checks (rust-lang/cargo#17385)
@weihanglo
weihanglo deleted the 198 branch August 27, 2026 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-registries Area: registries

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants