tree d6ed90135564444546740a5ba24ff10642984f1c
parent de9c53c655b96e3336ec49e50464e9c7b16b7d41
author Jonathan Amsterdam <jba@google.com> 1664549094 -0400
committer Jonathan Amsterdam <jba@google.com> 1664569592 +0000

slog: faster Record.Clone

Record.Clone no longer copies the Record's []Attr.

Previously, Record.Clone made a copy of the Record's Attr slice.
Since a Record's list of Attrs can only be appended to, we don't need
to copy the slice, only clip it (setting capacity to length). We
are effectively doing copy-on-write.

After this change, is Clone still the right name?

Change-Id: I10d86b2b5b17e503224fc9f9bb5c07e70e8fca36
Reviewed-on: https://go-review.googlesource.com/c/exp/+/436756
Run-TryBot: Jonathan Amsterdam <jba@google.com>
Reviewed-by: Alan Donovan <adonovan@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>