Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
alloc: Make deriving more friendly with Arc
This adds impls of Eq/Ord/PartialEq/PartialOrd/Show/Default to Arc<T>, and it also removes the `Send + Sync` bound on the `Clone` impl of Arc to make it more deriving-friendly. The `Send + Sync` requirement is still enforce on construction, of course!
- Loading branch information
1 parent
823f805
commit 35ad00d
Showing
2 changed files
with
54 additions
and
20 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
35ad00d
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 aturon
at alexcrichton@35ad00d
35ad00d
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/deriving-arc = 35ad00d into auto
35ad00d
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/deriving-arc = 35ad00d merged ok, testing candidate = faed648
35ad00d
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/1985
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/1984
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/1977
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/1984
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/1979
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/1978
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/1978
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/1982
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/1975
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/1975
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1978
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/1634
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/1628
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/1636
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/474
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/469
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-c/builds/462
35ad00d
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 = faed648