Uh oh!
There was an error while loading. Please reload this page.
Remove unused macros from the codebase - #41934
Merged
Merged
Conversation
rust-highfive
commented
May 12, 2017
Contributor
r? @arielb1 (rust_highfive has picked a reviewer for you, use r? to override) |
est31
commented
May 12, 2017
MemberAuthor
I want to get this through separate review as its easy and simple, and to not burden @jseyfried . This change does block the other PR though, as if the lint gets added, we emit warnings for each of these macros and all of the crates have |
Removes unused macros from: * libcore * libcollections The last use of these two macros was removed in commit b64c9d5 when the char_range_at_reverse function was been removed. * librustc_errors Their last use was removed by commits 2f2c3e1 and 11dc974. * libsyntax_ext * librustc_trans Also, put the otry macro in back/msvc/mod.rs under the same cfg argument as the places that use it.
nagisa
commented
May 12, 2017
Member
@bors r+ |
bors
commented
May 12, 2017
Collaborator
📌 Commit 80891f6 has been approved by |
Mark-Simulacrum added a commit
to Mark-Simulacrum/rust
that referenced
this pull request
May 12, 2017
Remove unused macros from the codebase Thanks to the lint I've implemented in rust-lang#41907 I've found some unused macros inside the rustc codebase.
Mark-Simulacrum added a commit
to Mark-Simulacrum/rust
that referenced
this pull request
May 12, 2017
Remove unused macros from the codebase Thanks to the lint I've implemented in rust-lang#41907 I've found some unused macros inside the rustc codebase.
Mark-Simulacrum added a commit
to Mark-Simulacrum/rust
that referenced
this pull request
May 12, 2017
Remove unused macros from the codebase Thanks to the lint I've implemented in rust-lang#41907 I've found some unused macros inside the rustc codebase.
bors added a commit
that referenced
this pull request
May 12, 2017
Mark-Simulacrum added a commit
to Mark-Simulacrum/rust
that referenced
this pull request
May 13, 2017
Remove unused macros from the codebase Thanks to the lint I've implemented in rust-lang#41907 I've found some unused macros inside the rustc codebase.
eddyb
reviewed
May 13, 2017
| // option. This file may not be copied, modified, or distributed | ||
| // except according to those terms. | ||
| macro_rules! unpack_datum { |
Contributor
There was a problem hiding this comment.
Ahahahahahhaahahaha I didn't even realize this file existed.
eddyb
commented
May 13, 2017
Contributor
@bors rollup |
bors added a commit
that referenced
this pull request
May 13, 2017
frewsxcv added a commit
to frewsxcv/rust
that referenced
this pull request
May 16, 2017
Add lint for unused macros Addresses parts of rust-lang#34938, to add a lint for unused macros. We now output warnings by default when we encounter a macro that we didn't use for expansion. Issues to be resolved before this PR is ready for merge: - [x] fix the NodeId issue described above - [x] remove all unused macros from rustc and the libraries or set `#[allow(unused_macros)]` next to them if they should be kept for some reason. This is needed for successful boostrap and bors to accept the PR. -> rust-lang#41934 - [x] ~~implement the full extent of rust-lang#34938, that means the macro match arm checking as well.~~ *let's not do this for now*
Mark-Simulacrum added a commit
to Mark-Simulacrum/rust
that referenced
this pull request
May 16, 2017
Add lint for unused macros Addresses parts of rust-lang#34938, to add a lint for unused macros. We now output warnings by default when we encounter a macro that we didn't use for expansion. Issues to be resolved before this PR is ready for merge: - [x] fix the NodeId issue described above - [x] remove all unused macros from rustc and the libraries or set `#[allow(unused_macros)]` next to them if they should be kept for some reason. This is needed for successful boostrap and bors to accept the PR. -> rust-lang#41934 - [x] ~~implement the full extent of rust-lang#34938, that means the macro match arm checking as well.~~ *let's not do this for now*
bors added a commit
that referenced
this pull request
May 16, 2017
Add lint for unused macros Addresses parts of #34938, to add a lint for unused macros. We now output warnings by default when we encounter a macro that we didn't use for expansion. Issues to be resolved before this PR is ready for merge: - [x] fix the NodeId issue described above - [x] remove all unused macros from rustc and the libraries or set `#[allow(unused_macros)]` next to them if they should be kept for some reason. This is needed for successful boostrap and bors to accept the PR. -> #41934 - [x] ~~implement the full extent of #34938, that means the macro match arm checking as well.~~ *let's not do this for now*
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Thanks to the lint I've implemented in #41907 I've found some unused macros inside the rustc codebase.