Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
syntax: Permit visibility on tuple fields
This change is in preparation for #8122. Nothing is currently done with these visibility qualifiers, they are just parsed and accepted by the compiler. RFC: 0004-private-fields
- Loading branch information
1 parent
104aaa4
commit 7de4841
Showing
10 changed files
with
30 additions
and
21 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
7de4841
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 brson
at alexcrichton@7de4841
7de4841
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 alexcrichton/rust/flip-some-defaults = 7de4841 into auto
7de4841
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.
alexcrichton/rust/flip-some-defaults = 7de4841 merged ok, testing candidate = bbdffa48
7de4841
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.
some tests failed:
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/4902
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/4003
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/5002
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/5004
7de4841
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 brson
at alexcrichton@7de4841
7de4841
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 alexcrichton/rust/flip-some-defaults = 7de4841 into auto
7de4841
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.
alexcrichton/rust/flip-some-defaults = 7de4841 merged ok, testing candidate = a27bfcac
7de4841
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.
some tests failed:
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/4900
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/3996
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/5003
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/4090
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/4098
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/5005
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/4090
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/4095
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/4161
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1894
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/5001
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/4109
7de4841
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 brson
at alexcrichton@7de4841
7de4841
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 alexcrichton/rust/flip-some-defaults = 7de4841 into auto
7de4841
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.
alexcrichton/rust/flip-some-defaults = 7de4841 merged ok, testing candidate = c83994e
7de4841
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/4905
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/4903
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/3999
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/4006
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/5005
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/4092
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/4100
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/5007
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/4092
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/4097
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/4163
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1896
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/5003
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/4103
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/4111
success: http://buildbot.rust-lang.org/builders/auto-bsd-64-opt/builds/4768
7de4841
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 = c83994e