Skip to content
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

Issue deprecation warnings for safe accesses to extern statics #36173

Merged
merged 1 commit into from
Sep 9, 2016

Conversation

petrochenkov
Copy link
Contributor

@petrochenkov petrochenkov commented Aug 31, 2016

Fixes #35112
cc #36247

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@@ -207,6 +207,10 @@ pub fn register_builtins(store: &mut lint::LintStore, sess: Option<&Session>) {
id: LintId::of(LIFETIME_UNDERSCORE),
reference: "RFC 1177 <https://github.com/rust-lang/rfcs/pull/1177>",
},
FutureIncompatibleInfo {
id: LintId::of(SAFE_EXTERN_STATICS),
reference: "issue 35112 <https://github.com/rust-lang/rust/issues/35112>",
Copy link
Contributor

Choose a reason for hiding this comment

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

the new policy is to make a separate tracking issue, following roughly this template; if you could update with an issue like that, it'd be great.

@petrochenkov
Copy link
Contributor Author

@nikomatsakis
Updated.

@nikomatsakis
Copy link
Contributor

Looks great. =)

@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Sep 8, 2016

📌 Commit ef489fc has been approved by nikomatsakis

@bors
Copy link
Contributor

bors commented Sep 8, 2016

⌛ Testing commit ef489fc with merge 4391d8e...

@bors
Copy link
Contributor

bors commented Sep 8, 2016

💔 Test failed - auto-win-msvc-64-opt-rustbuild

@petrochenkov
Copy link
Contributor Author

@bors r=nikomatsakis

@bors
Copy link
Contributor

bors commented Sep 8, 2016

📌 Commit aadbcff has been approved by nikomatsakis

@bors
Copy link
Contributor

bors commented Sep 8, 2016

⌛ Testing commit aadbcff with merge 3781956...

bors added a commit that referenced this pull request Sep 8, 2016
Issue deprecation warnings for safe accesses to extern statics

Fixes #35112
cc #36247
@bors bors merged commit aadbcff into rust-lang:master Sep 9, 2016
@bluss bluss added the relnotes Marks issues that should be documented in the release notes of the next release. label Sep 11, 2016
@petrochenkov petrochenkov deleted the unstat branch September 21, 2016 19:59
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.

5 participants