-
Notifications
You must be signed in to change notification settings - Fork 12.5k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
change to new trait style for method field refs
Per @pnkfelix 's suggestion, using a trait to make these field accesses more readable (and vastly more similar to the original code. oops fix new ast_map fix
- Loading branch information
1 parent
154ca08
commit ca05828
Showing
17 changed files
with
99 additions
and
113 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
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
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
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
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
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
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
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
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
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
Oops, something went wrong.
ca05828
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 alexcrichton
at jbclements@ca05828
ca05828
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 jbclements/rust/method-field-cleanup = ca05828 into auto
ca05828
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.
jbclements/rust/method-field-cleanup = ca05828 merged ok, testing candidate = efbbb51
ca05828
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/356
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/357
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/356
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/357
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/359
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/360
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/359
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/365
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/360
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/359
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/362
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/360
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/360
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/361
ca05828
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 = efbbb51