Skip to content

Clarify "owned data" in E0515.md - #138508

Merged
bors merged 1 commit into
rust-lang:masterfrom
hkBst:patch-3
Mar 18, 2025
Merged

Clarify "owned data" in E0515.md#138508
bors merged 1 commit into
rust-lang:masterfrom
hkBst:patch-3

Conversation

@hkBst

@hkBsthkBst commented Mar 14, 2025

Copy link
Copy Markdown
Member

This clarifies the explanation of why this is not allowed and also what to do instead.

Fixes#62071

PS There was suggestion of adding a link to the book. I did not yet do that, but if desired that could be added.

@rustbot

Copy link
Copy Markdown
Collaborator

r? @wesleywiser

rustbot has assigned @wesleywiser.
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

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Mar 14, 2025
@rust-log-analyzer

This comment has been minimized.

This clarifies the explanation of why this is not allowed and also what to do instead.
Fixes 62071
PS There was suggestion of adding a link to the book. I did not yet do that, but if desired that could be added.
@wesleywiser

Copy link
Copy Markdown
Member

Thanks @hkBst!

@bors r+ rollup

@bors

bors commented Mar 17, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit 50c659f has been approved by wesleywiser

It is now in the queue for this repository.

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 17, 2025
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 18, 2025
…iaskrgr
Rollup of 7 pull requests
Successful merges:
- rust-lang#138384 (Move `hir::Item::ident` into `hir::ItemKind`.)
- rust-lang#138508 (Clarify "owned data" in E0515.md)
- rust-lang#138531 (Store test diffs in job summaries and improve analysis formatting)
- rust-lang#138533 (Only use `DIST_TRY_BUILD` for try jobs that were not selected explicitly)
- rust-lang#138556 (Fix ICE: attempted to remap an already remapped filename)
- rust-lang#138608 (rustc_target: Add target feature constraints for LoongArch)
- rust-lang#138619 (Flatten `if`s in `rustc_codegen_ssa`)
r? `@ghost`
`@rustbot` modify labels: rollup
jhpratt added a commit to jhpratt/rust that referenced this pull request Mar 18, 2025
Clarify "owned data" in E0515.md
This clarifies the explanation of why this is not allowed and also what to do instead.
Fixesrust-lang#62071
PS There was suggestion of adding a link to the book. I did not yet do that, but if desired that could be added.
@jhprattjhpratt mentioned this pull request Mar 18, 2025
jhpratt added a commit to jhpratt/rust that referenced this pull request Mar 18, 2025
Clarify "owned data" in E0515.md
This clarifies the explanation of why this is not allowed and also what to do instead.
Fixesrust-lang#62071
PS There was suggestion of adding a link to the book. I did not yet do that, but if desired that could be added.
@jhprattjhpratt mentioned this pull request Mar 18, 2025
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 18, 2025
Rollup of 7 pull requests
Successful merges:
- rust-lang#136320 (exit: document interaction with C)
- rust-lang#138301 (Implement `read_buf` for Hermit)
- rust-lang#138508 (Clarify "owned data" in E0515.md)
- rust-lang#138556 (Fix ICE: attempted to remap an already remapped filename)
- rust-lang#138569 (rustdoc-json: Add tests for `#[repr(...)]`)
- rust-lang#138608 (rustc_target: Add target feature constraints for LoongArch)
- rust-lang#138619 (Flatten `if`s in `rustc_codegen_ssa`)
r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 18, 2025
…iaskrgr
Rollup of 7 pull requests
Successful merges:
- rust-lang#138384 (Move `hir::Item::ident` into `hir::ItemKind`.)
- rust-lang#138508 (Clarify "owned data" in E0515.md)
- rust-lang#138531 (Store test diffs in job summaries and improve analysis formatting)
- rust-lang#138533 (Only use `DIST_TRY_BUILD` for try jobs that were not selected explicitly)
- rust-lang#138556 (Fix ICE: attempted to remap an already remapped filename)
- rust-lang#138608 (rustc_target: Add target feature constraints for LoongArch)
- rust-lang#138619 (Flatten `if`s in `rustc_codegen_ssa`)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 01062ba into rust-lang:masterMar 18, 2025
@rustbotrustbot added this to the 1.87.0 milestone Mar 18, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Mar 18, 2025
Rollup merge of rust-lang#138508 - hkBst:patch-3, r=wesleywiser
Clarify "owned data" in E0515.md
This clarifies the explanation of why this is not allowed and also what to do instead.
Fixesrust-lang#62071
PS There was suggestion of adding a link to the book. I did not yet do that, but if desired that could be added.
@hkBst
hkBst deleted the patch-3 branch March 18, 2025 10:18
flip1995 pushed a commit to flip1995/rust that referenced this pull request Mar 20, 2025
…iaskrgr
Rollup of 7 pull requests
Successful merges:
- rust-lang#138384 (Move `hir::Item::ident` into `hir::ItemKind`.)
- rust-lang#138508 (Clarify "owned data" in E0515.md)
- rust-lang#138531 (Store test diffs in job summaries and improve analysis formatting)
- rust-lang#138533 (Only use `DIST_TRY_BUILD` for try jobs that were not selected explicitly)
- rust-lang#138556 (Fix ICE: attempted to remap an already remapped filename)
- rust-lang#138608 (rustc_target: Add target feature constraints for LoongArch)
- rust-lang#138619 (Flatten `if`s in `rustc_codegen_ssa`)
r? `@ghost`
`@rustbot` modify labels: rollup
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

E0515 searching internet for the phrase "owned value" does not give helpful results

5 participants

@hkBst@rustbot@rust-log-analyzer@wesleywiser@bors