-
Notifications
You must be signed in to change notification settings - Fork 12.9k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
item_impl holds an Option<> to the trait ref, not a list of trait ref…
…s. Therefore, we should not iterate over it.
- Loading branch information
Showing
1 changed file
with
6 additions
and
5 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
40d8a12
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 nrc@40d8a12
40d8a12
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 nick29581/rust/visit_trait_option = 40d8a12 into auto
40d8a12
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.
nick29581/rust/visit_trait_option = 40d8a12 merged ok, testing candidate = 5a6ca45
40d8a12
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/3497
success: http://buildbot.rust-lang.org/builders/auto-mac-32-nopt-c/builds/1339
success: http://buildbot.rust-lang.org/builders/auto-mac-32-nopt-t/builds/1343
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3499
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/2600
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/2605
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3515
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/2603
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/2609
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3517
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/2603
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/2608
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/2677
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/3505
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/2607
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2611
success: http://buildbot.rust-lang.org/builders/auto-bsd-64-opt/builds/3278
40d8a12
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 = 5a6ca45