diff options
| author | bors <bors@rust-lang.org> | 2015-01-26 13:01:00 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2015-01-26 13:01:00 +0000 |
| commit | 977c44ade09450016d9c95dea245fd4464a509e9 (patch) | |
| tree | adea995c7831ebae76bca46e243cbd124f53626b /src/libstd/sys/unix/stack_overflow.rs | |
| parent | 59dcba5d14e5eada0a5e7eb0cad5efc7acd1d0e3 (diff) | |
| parent | c384ee18fcb55274682e8a9a24608bfc825bedce (diff) | |
| download | rust-977c44ade09450016d9c95dea245fd4464a509e9.tar.gz rust-977c44ade09450016d9c95dea245fd4464a509e9.zip | |
Auto merge of #21401 - kballard:optimize-shrink-to-fit, r=nikomatsakis
Don't reallocate when capacity is already equal to length
`Vec::shrink_to_fit()` may be called on vectors that are already the
correct length. Calling out to `reallocate()` in this case is a bad idea
because there is no guarantee that `reallocate()` won't allocate a new
buffer anyway, and based on performance seen in external benchmarks, it
seems likely that it is in fact reallocating a new buffer.
Before:
test string::tests::bench_exact_size_shrink_to_fit ... bench: 45 ns/iter (+/- 2)
After:
test string::tests::bench_exact_size_shrink_to_fit ... bench: 26 ns/iter (+/- 1)
Diffstat (limited to 'src/libstd/sys/unix/stack_overflow.rs')
0 files changed, 0 insertions, 0 deletions
