Skip to content

Tweak "object unsafe" errors #78002

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Oct 21, 2020
Merged

Tweak "object unsafe" errors #78002

merged 2 commits into from
Oct 21, 2020

Conversation

estebank
Copy link
Contributor

@estebank estebank commented Oct 16, 2020 •
edited
Loading

CC #77598.

@rust-highfive
Copy link
Contributor

r? @oli-obk

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 16, 2020
@jyn514 jyn514 added A-diagnostics Area: Messages for errors, warnings, and lints T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Oct 16, 2020
oli-obk
oli-obk reviewed Oct 16, 2020
View reviewed changes
oli-obk
oli-obk reviewed Oct 16, 2020
View reviewed changes
oli-obk
oli-obk reviewed Oct 16, 2020
View reviewed changes
@bors

This comment has been minimized.

Sign in to view
@oli-obk
Copy link
Contributor

oli-obk commented Oct 20, 2020

r=me (needs rebase)

@estebank
Copy link
Contributor Author

@bors r=oli-obk

@bors
Copy link
Collaborator

bors commented Oct 20, 2020

📌 Commit 88f5e11 has been approved by oli-obk

@bors bors 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 Oct 20, 2020
@JohnTitor JohnTitor mentioned this pull request Oct 21, 2020
Merged
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 21, 2020
@bors
Auto merge of rust-lang#78178 - JohnTitor:rollup-dslazzj, r=JohnTitor
1935645 
Rollup of 7 pull requests

Successful merges:

 - rust-lang#77726 (Add Pin::static_ref, static_mut.)
 - rust-lang#78002 (Tweak "object unsafe" errors)
 - rust-lang#78056 (BTreeMap: split off most code of remove and split_off)
 - rust-lang#78063 (Improve wording of "cannot multiply" type error)
 - rust-lang#78094 (rustdoc: Show the correct source filename in page titles, without `.html`)
 - rust-lang#78101 (fix static_ptr_ty for foreign statics)
 - rust-lang#78118 (Inline const followups)

Failed merges:

r? `@ghost`
@bors bors merged commit 9583029 into rust-lang:master Oct 21, 2020
@rustbot rustbot added this to the 1.49.0 milestone Oct 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oli-obk oli-obk oli-obk left review comments

Assignees

@oli-obk oli-obk

Labels
A-diagnostics Area: Messages for errors, warnings, and lints S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
1.49.0
Development

Successfully merging this pull request may close these issues.
6 participants
@estebank @rust-highfive @bors @oli-obk @jyn514 @rustbot