Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
rustc: Add deprecation/renaming support for lints
Since a large number of lints are being renamed for RFC 344, this commit adds some basic deprecation/renaming functionality to the pluggable lint system. It allows a simple mapping of old to new names, and can warn when old names are being used. This change needs to be rolled out in stages. In this commit, the deprecation warning is commented out, but the old name is forwarded to the new one. Once the commit lands and we have generated a new snapshot of the compiler, we can add the deprecation warning and rename all uses of the lints in the rust codebase.
- Loading branch information
Showing
1 changed file
with
70 additions
and
6 deletions.
There are no files selected for viewing
This file contains 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
31b7d64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
saw approval from pcwalton
at aturon@31b7d64
31b7d64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merging aturon/rust/lint-conventions = 31b7d64 into auto
31b7d64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aturon/rust/lint-conventions = 31b7d64 merged ok, testing candidate = ea8a5df
31b7d64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/1790
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/1787
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/1781
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/1787
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/1784
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/1783
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/1783
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/1787
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/1781
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/1780
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1783
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/1439
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/1438
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/1440
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/277
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/278
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-c/builds/270
31b7d64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fast-forwarding master to auto = ea8a5df