Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
test: make the sort failure-safety test unsafeless and more obvious.
Previously it had some uninituitive conditionals due to the interaction with the Rand construction and Clone reinitialisation to create sequential identifying numbers. This replaces all that with just constructing the DropCounters with the appropriate identifiers.
- Loading branch information
Showing
1 changed file
with
43 additions
and
31 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
32513b0
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 huonw@32513b0
32513b0
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 huonw/rust/adjust-vec-sort-test = 32513b0 into auto
32513b0
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.
huonw/rust/adjust-vec-sort-test = 32513b0 merged ok, testing candidate = 5201bf1
32513b0
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/1791
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/1788
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/1782
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/1788
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/1785
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/1784
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/1784
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/1788
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/1782
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/1781
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1784
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/1440
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/1439
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/1441
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/278
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/279
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-c/builds/271
32513b0
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 = 5201bf1