diff options
| author | Keith Randall <khr@golang.org> | 2020-09-24 19:26:33 -0700 |
|---|---|---|
| committer | Keith Randall <khr@golang.org> | 2020-09-25 03:59:54 +0000 |
| commit | 2333c6299f340a5f76a73a4fec6db23ffa388e97 (patch) | |
| tree | f389380ff340902987186e7fff7ac6f7f0a3bddb /src/runtime | |
| parent | fa04d488bd54b8fdd78cc9bcc6d90de4bf5f8efb (diff) | |
| download | go-2333c6299f340a5f76a73a4fec6db23ffa388e97.tar.xz | |
runtime: use old capacity to decide on append growth regime
We grow the backing store on append by 2x for small sizes and 1.25x
for large sizes. The threshold we use for picking the growth factor
used to depend on the old length, not the old capacity. That's kind of
unfortunate, because then doing append(s, 0, 0) and append(append(s,
0), 0) do different things. (If s has one more spot available, then
the former expression chooses its growth based on len(s) and the
latter on len(s)+1.) If we instead use the old capacity, we get more
consistent behavior. (Both expressions use len(s)+1 == cap(s) to
decide.)
Fixes #41239
Change-Id: I40686471d256edd72ec92aef973a89b52e235d4b
Reviewed-on: https://go-review.googlesource.com/c/go/+/257338
Trust: Keith Randall <khr@golang.org>
Trust: Josh Bleecher Snyder <josharian@gmail.com>
Run-TryBot: Keith Randall <khr@golang.org>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Josh Bleecher Snyder <josharian@gmail.com>
Diffstat (limited to 'src/runtime')
| -rw-r--r-- | src/runtime/slice.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/runtime/slice.go b/src/runtime/slice.go index 82a45c78a9..c0647d95a0 100644 --- a/src/runtime/slice.go +++ b/src/runtime/slice.go @@ -146,7 +146,7 @@ func growslice(et *_type, old slice, cap int) slice { if cap > doublecap { newcap = cap } else { - if old.len < 1024 { + if old.cap < 1024 { newcap = doublecap } else { // Check 0 < newcap to detect overflow |