Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
std: Change the behavior of
reserve
for HashMap.
HashMap's `reserve` method now takes as an argument the *extra* space to reserve. [breaking-change]
- Loading branch information
Showing
1 changed file
with
17 additions
and
6 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
b82624b
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 Gankro
at pczarn@b82624b
b82624b
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 pczarn/rust/hash_map-explicit-shrinking = b82624b into auto
b82624b
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.
pczarn/rust/hash_map-explicit-shrinking = b82624b merged ok, testing candidate = 680d1fb4
b82624b
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:
failure: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/2515
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/2518
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/2508
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/2515
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/2509
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/2508
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/2508
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/2512
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/2505
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/2504
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2507
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2169
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2165
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/2171
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1008
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1002
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-c/builds/998
b82624b
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 Gankro
at pczarn@b82624b
b82624b
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 pczarn/rust/hash_map-explicit-shrinking = b82624b into auto
b82624b
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.
pczarn/rust/hash_map-explicit-shrinking = b82624b merged ok, testing candidate = 207a508
b82624b
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/2552
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/2549
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/2542
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/2549
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/2543
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/2542
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/2542
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/2546
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/2539
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/2538
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2541
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2203
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2199
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/2205
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1042
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1036
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-c/builds/1032
b82624b
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 = 207a508