Skip to content

Remove unused code from librustc_errors #43019

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
Jul 8, 2017

Conversation

kevinmehall
Copy link
Contributor

While extracting librustc_errors into a reusable library, I noticed some obsolete code that the dead_code warning missed because it was marked pub but not used elsewhere.

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@aidanhs aidanhs added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 5, 2017
@aidanhs
Copy link
Member

aidanhs commented Jul 5, 2017

Thanks for the PR @kevinmehall, we'll check in to make sure @nikomatsakis or another reviewer gets to this soon!

@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 6, 2017

📌 Commit e863d26 has been approved by nikomatsakis

@bors
Copy link
Collaborator

bors commented Jul 6, 2017

🔒 Merge conflict

@kevinmehall
Copy link
Contributor Author

@nikomatsakis Rebased

@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 7, 2017

📌 Commit 17bd76a has been approved by nikomatsakis

@nikomatsakis
Copy link
Contributor

@bors rollup

@bors
Copy link
Collaborator

bors commented Jul 8, 2017

⌛ Testing commit 17bd76a with merge b695889...

bors added a commit that referenced this pull request Jul 8, 2017
Remove unused code from librustc_errors

While extracting librustc_errors into a [reusable library](https://github.com/kevinmehall/codemap-diagnostic), I noticed some obsolete code that the `dead_code` warning missed because it was marked `pub` but not used elsewhere.
@bors
Copy link
Collaborator

bors commented Jul 8, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: nikomatsakis
Pushing b695889 to master...

@bors bors merged commit 17bd76a into rust-lang:master Jul 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants