Skip to content

Remove mutability from Def::Static #60124

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 4 commits into from
Apr 21, 2019
Merged

Conversation

petrochenkov
Copy link
Contributor

@petrochenkov petrochenkov commented Apr 19, 2019

Querify TyCtxt::is_static.
Use Mutability instead of bool in foreign statics in AST/HIR.

cc #60110
r? @eddyb

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 19, 2019
@petrochenkov petrochenkov changed the title [WIP] Remove mutability from Def::Static Remove mutability from Def::Static Apr 21, 2019
@petrochenkov
Copy link
Contributor Author

Updated.

@eddyb
Copy link
Member

eddyb commented Apr 21, 2019

@bors r+

@bors
Copy link
Collaborator

bors commented Apr 21, 2019

📌 Commit 4eb94b4 has been approved by eddyb

@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 Apr 21, 2019
bors added a commit that referenced this pull request Apr 21, 2019
Remove mutability from `Def::Static`

Querify `TyCtxt::is_static`.
Use `Mutability` instead of bool in foreign statics in AST/HIR.

cc #60110
r? @eddyb
@bors
Copy link
Collaborator

bors commented Apr 21, 2019

⌛ Testing commit 4eb94b4 with merge 31a75a1...

@bors
Copy link
Collaborator

bors commented Apr 21, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: eddyb
Pushing 31a75a1 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Apr 21, 2019
@bors bors merged commit 4eb94b4 into rust-lang:master Apr 21, 2019
@rust-highfive
Copy link
Contributor

📣 Toolstate changed by #60124!

Tested on commit 31a75a1.
Direct link to PR: #60124

💔 clippy-driver on windows: test-pass → build-fail (cc @Manishearth @llogiq @mcarton @oli-obk @phansch, @rust-lang/infra).
💔 clippy-driver on linux: test-pass → build-fail (cc @Manishearth @llogiq @mcarton @oli-obk @phansch, @rust-lang/infra).
💔 rls on windows: test-pass → build-fail (cc @Xanewok, @rust-lang/infra).
💔 rls on linux: test-pass → build-fail (cc @Xanewok, @rust-lang/infra).

rust-highfive added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Apr 21, 2019
Tested on commit rust-lang/rust@31a75a1.
Direct link to PR: <rust-lang/rust#60124>

💔 clippy-driver on windows: test-pass → build-fail (cc @Manishearth @llogiq @mcarton @oli-obk @phansch, @rust-lang/infra).
💔 clippy-driver on linux: test-pass → build-fail (cc @Manishearth @llogiq @mcarton @oli-obk @phansch, @rust-lang/infra).
💔 rls on windows: test-pass → build-fail (cc @Xanewok, @rust-lang/infra).
💔 rls on linux: test-pass → build-fail (cc @Xanewok, @rust-lang/infra).
Xanewok added a commit to Xanewok/rust-clippy that referenced this pull request Apr 21, 2019
Xanewok added a commit to Xanewok/rust-clippy that referenced this pull request Apr 21, 2019
bors added a commit to rust-lang/rust-clippy that referenced this pull request Apr 21, 2019
rustup: Use newly-introduced mutability query for statics

Fixes fallout from rust-lang/rust#60124.
cc rust-lang/rust#60154

changelog: none
@Xanewok Xanewok mentioned this pull request Apr 21, 2019
Xanewok added a commit to Xanewok/rust that referenced this pull request Apr 21, 2019
Fixes fallout from rust-lang#60124.
Closes rust-lang#60154.
bors added a commit that referenced this pull request Apr 21, 2019
@petrochenkov petrochenkov deleted the stanomut branch June 5, 2019 16:33
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request May 5, 2020
Fixes fallout from rust-lang/rust#60124.
Closes #60154.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants