slices, for loop copy, min/max update

core/block_test.go

@@ -171,8 +171,7 @@ func TestBlockHash(t *testing.T) {
 		},
 	}
 
-	for _, testcase := range tests {
-		tc := testcase
+	for _, tc := range tests {
 		t.Run(tc.name, func(t *testing.T) {
 			t.Parallel()
 			client := feeder.NewTestClient(t, &tc.chain)

core/state.go

@@ -6,7 +6,7 @@
 	"errors"
 	"fmt"
 	"runtime"
-	"sort"
+	"slices"
 
 	"github.com/NethermindEth/juno/core/crypto"
 	"github.com/NethermindEth/juno/core/felt"
@@ -400,9 +400,7 @@
 	for key := range diffs {
 		keys = append(keys, key)
 	}
-	sort.SliceStable(keys, func(i, j int) bool {
-		return len(diffs[keys[i]]) > len(diffs[keys[j]])
-	})
+	slices.SortStableFunc(keys, func(a, b felt.Felt) int { return a.Cmp(&b) })
 
 	// update per-contract storage Tries concurrently
 	contractUpdaters := pool.NewWithResults[*bufferedTransactionWithAddress]().WithErrors().WithMaxGoroutines(runtime.GOMAXPROCS(0))
@@ -423,7 +421,7 @@
 	}
 
 	// we sort bufferedTxns in ascending contract address order to achieve an additional speedup
 	sort.Slice(bufferedTxns, func(i, j int) bool {
 		return bufferedTxns[i].addr.Cmp(bufferedTxns[j].addr) < 0
 	})
 

core/transaction.go

@@ -570,8 +570,7 @@ func l1HandlerTransactionHash(l *L1HandlerTransaction, n *utils.Network) (*felt.
 }
 
 func deployAccountTransactionHash(d *DeployAccountTransaction, n *utils.Network) (*felt.Felt, error) {
-	callData := []*felt.Felt{d.ClassHash, d.ContractAddressSalt}
-	callData = append(callData, d.ConstructorCallData...)
+	callData := append([]*felt.Felt{d.ClassHash, d.ContractAddressSalt}, d.ConstructorCallData...)
 	// There is no version 0 for deploy account
 	switch {
 	case d.Version.Is(1):
@@ -698,9 +697,8 @@ func ParseBlockVersion(protocolVersion string) (*semver.Version, error) {
 	}
 
 	sep := "."
-	digits := strings.Split(protocolVersion, sep)
 	// pad with 3 zeros in case version has less than 3 digits
-	digits = append(digits, []string{"0", "0", "0"}...)
+	digits := append(strings.Split(protocolVersion, sep), []string{"0", "0", "0"}...)
 
 	// get first 3 digits only
 	return semver.NewVersion(strings.Join(digits[:3], sep))

db/pebble/iterator.go

@@ -1,6 +1,8 @@
 package pebble
 
 import (
+	"slices"
+
 	"github.com/NethermindEth/juno/db"
 	"github.com/cockroachdb/pebble"
 )
@@ -23,8 +25,7 @@ func (i *iterator) Key() []byte {
 	if key == nil {
 		return nil
 	}
-	buf := make([]byte, len(key))
-	copy(buf, key)
+	buf := slices.Clone(key)
 	return buf
 }
 
@@ -34,8 +35,7 @@ func (i *iterator) Value() ([]byte, error) {
 	if err != nil || val == nil {
 		return nil, err
 	}
-	buf := make([]byte, len(val))
-	copy(buf, val)
+	buf := slices.Clone(val)
 	return buf, nil
 }
 

l1/l1_pkg_test.go

@@ -330,7 +330,6 @@ func TestClient(t *testing.T) {
 	}
 
 	for _, tt := range tests {
-		tt := tt
 		t.Run(tt.description, func(t *testing.T) {
 			t.Parallel()
 

rpc/block_test.go

@@ -52,7 +52,6 @@ func TestBlockId(t *testing.T) {
 		},
 	}
 	for name, test := range tests {
-		test := test
 		t.Run(name, func(t *testing.T) {
 			t.Parallel()
 			var blockID rpc.BlockID
@@ -81,7 +80,6 @@ func TestBlockId(t *testing.T) {
 		},
 	}
 	for name, test := range failingTests {
-		test := test
 		t.Run(name, func(t *testing.T) {
 			t.Parallel()
 			var blockID rpc.BlockID

sync/sync.go

@@ -298,11 +298,7 @@ func (s *Synchronizer) syncBlocks(syncCtx context.Context) {
 }
 
 func maxWorkers() int {
-	m, mProcs := 16, runtime.GOMAXPROCS(0)
-	if mProcs > m {
-		return m
-	}
-	return mProcs
+	return min(16, runtime.GOMAXPROCS(0)) //nolint:mnd
 }
 
 func (s *Synchronizer) setupWorkers() (*stream.Stream, *stream.Stream) {

upgrader/upgrader_test.go

@@ -102,7 +102,6 @@ func TestUpgrader(t *testing.T) {
 	}
 
 	for description, test := range tests {
-		test := test
 		t.Run(description, func(t *testing.T) {
 			t.Parallel()
 			srv := httptest.NewServer(http.HandlerFunc(func(rw http.ResponseWriter, req *http.Request) {

vm/vm.go

@@ -344,7 +344,7 @@ func (v *vm) Execute(txns []core.Transaction, declaredClasses []core.Class, paid
 }
 
 func marshalTxnsAndDeclaredClasses(txns []core.Transaction, declaredClasses []core.Class) (json.RawMessage, json.RawMessage, error) { //nolint:lll
-	txnJSONs := []json.RawMessage{}
+	txnJSONs := make([]json.RawMessage, 0, len(txns))
 	for _, txn := range txns {
 		txnJSON, err := marshalTxn(txn)
 		if err != nil {
@@ -353,7 +353,7 @@ func marshalTxnsAndDeclaredClasses(txns []core.Transaction, declaredClasses []co
 		txnJSONs = append(txnJSONs, txnJSON)
 	}
 
-	classJSONs := []json.RawMessage{}
+	classJSONs := make([]json.RawMessage, 0, len(declaredClasses))
 	for _, declaredClass := range declaredClasses {
 		declaredClassJSON, cErr := marshalClassInfo(declaredClass)
 		if cErr != nil {