Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
std: Move MemoryMap fields to methods
If modified, you can safely unmap arbitrary memory. These fields are not intended to be modified, so read-only accessors are the only ones that are provided. Closes #15478
- Loading branch information
1 parent
345886c
commit fb02d54
Showing
2 changed files
with
25 additions
and
17 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
fb02d54
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 cmr
at alexcrichton@fb02d54
fb02d54
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/issue-15478 = fb02d54 into auto
fb02d54
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/issue-15478 = fb02d54 merged ok, testing candidate = a5688ad
fb02d54
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/299
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/300
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/299
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/300
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/302
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/303
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/302
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/308
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/303
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/302
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/305
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/303
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/303
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/304
fb02d54
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 = a5688ad