Skip to content

use the *adjusted* callee type in effect checking #28790

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.

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

Merged
merged 1 commit into from
Oct 2, 2015

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented Oct 1, 2015

Fixes #28776

r? @eddyb

@@ -105,6 +105,7 @@ pub mod front {
}

pub mod middle {
pub mod expr_use_visitor; // STAGE0: increase glitch immunity
Copy link
Member

Choose a reason for hiding this comment

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

This is starting to get a little bit silly 😃...

@eddyb
Copy link
Member

eddyb commented Oct 1, 2015

@bors r+

@bors
Copy link
Collaborator

bors commented Oct 1, 2015

📌 Commit 0706ac0 has been approved by eddyb

@bors
Copy link
Collaborator

bors commented Oct 2, 2015

⌛ Testing commit 0706ac0 with merge 0ffc682...

bors added a commit that referenced this pull request Oct 2, 2015
@bors bors merged commit 0706ac0 into rust-lang:master Oct 2, 2015
@brson
Copy link
Contributor

brson commented Oct 2, 2015

What is the impact of this change? Please provide more context.

@brson brson added the relnotes Marks issues that should be documented in the release notes of the next release. label Oct 2, 2015
@arielb1
Copy link
Contributor Author

arielb1 commented Oct 2, 2015

@brson

Previous versions of rustc could allow calling an unsafe function outside of an unsafe block under conditions involving autoderef.

@brson
Copy link
Contributor

brson commented Oct 8, 2015

@arielb1 Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relnotes Marks issues that should be documented in the release notes of the next release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants