Skip to content

Cover str in the wide-pointer metadata rule#2288

Merged
traviscross merged 2 commits into
masterfrom
TC/cover-str-in-wide-pointer-metadata-rule
Jun 16, 2026
Merged

Cover str in the wide-pointer metadata rule#2288
traviscross merged 2 commits into
masterfrom
TC/cover-str-in-wide-pointer-metadata-rule

Conversation

@traviscross

@traviscross traviscross commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

The validity rule for the metadata of a wide reference, Box<T>, or raw pointer mentions dyn Trait and slice but had omitted str. Let's fix that.


I'm breaking this out from #2282 so that we can merge the prerequisites before considering the new lang guarantees.

This is stacked on #2286 and #2287 and those should be merged first.

cc @ehuss @RalfJung @Mark-Simulacrum

@rustbot rustbot added the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Jun 16, 2026
@traviscross traviscross force-pushed the TC/cover-str-in-wide-pointer-metadata-rule branch from 3a51e52 to 4565f40 Compare June 16, 2026 04:48

@ehuss ehuss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ehuss ehuss removed the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Jun 16, 2026
We refer to the unsized tail of a type, but we hadn't defined it.
Let's do that and link to the definition.
The validity rule for the metadata of a wide reference, `Box<T>`,
or raw pointer mentions `dyn Trait` and slice but had omitted
`str`.  Let's fix that.
@traviscross traviscross force-pushed the TC/cover-str-in-wide-pointer-metadata-rule branch from 4565f40 to 082d04d Compare June 16, 2026 19:05
@rustbot

rustbot commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different master 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.

@traviscross traviscross added this pull request to the merge queue Jun 16, 2026
Merged via the queue into master with commit 9a10260 Jun 16, 2026
6 checks passed
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.

3 participants