Merge pull request #97964 from sxllwx/avoid/alloc_array

modify the elements in the array directly without allocating a new array

Kubernetes-commit: c1336094a3ccaa497c8074c888f92d4093aa3360
This commit is contained in:
Kubernetes Publisher 2021-04-23 04:02:16 -07:00
commit 9c26abce64

View File

@ -373,13 +373,8 @@ func dedupDeltas(deltas Deltas) Deltas {
a := &deltas[n-1]
b := &deltas[n-2]
if out := isDup(a, b); out != nil {
// `a` and `b` are duplicates. Only keep the one returned from isDup().
// TODO: This extra array allocation and copy seems unnecessary if
// all we do to dedup is compare the new delta with the last element
// in `items`, which could be done by mutating `items` directly.
// Might be worth profiling and investigating if it is safe to optimize.
d := append(Deltas{}, deltas[:n-2]...)
return append(d, *out)
deltas[n-2] = *out
return deltas[:n-1]
}
return deltas
}