From 25dcb2e67cf8d9a494ab52e08ac5c79124f89bdf Mon Sep 17 00:00:00 2001 From: maskpp Date: Mon, 1 Jul 2024 22:23:36 +0800 Subject: [PATCH] avoid trigger memory reallocation of slice --- trie/utils/verkle.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/trie/utils/verkle.go b/trie/utils/verkle.go index 2a4a632d49..dfcca22024 100644 --- a/trie/utils/verkle.go +++ b/trie/utils/verkle.go @@ -18,6 +18,7 @@ package utils import ( "encoding/binary" + "slices" "sync" "github.com/crate-crypto/go-ipa/bandersnatch/fr" @@ -307,9 +308,7 @@ func pointToHash(evaluated *verkle.Point, suffix byte) []byte { // tree would be empty. To avoid this problem, use a little // endian commitment and chop the MSB. bytes := evaluated.Bytes() - for i := 0; i < 16; i++ { - bytes[31-i], bytes[i] = bytes[i], bytes[31-i] - } + slices.Reverse(bytes[:]) bytes[31] = suffix return bytes[:] } @@ -317,7 +316,8 @@ func pointToHash(evaluated *verkle.Point, suffix byte) []byte { func evaluateAddressPoint(address []byte) *verkle.Point { if len(address) < 32 { var aligned [32]byte - address = append(aligned[:32-len(address)], address...) + copy(aligned[32-len(address):], address) + address = aligned[:] } var poly [3]fr.Element