From f829928fb1ce355949c5cd631736f35f9c49168d Mon Sep 17 00:00:00 2001
From: Marius van der Wijden <m.vanderwijden@live.de>
Date: Fri, 14 Mar 2025 11:03:21 +0100
Subject: [PATCH] accounts/abi: only loop once

---
 accounts/abi/argument.go | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/accounts/abi/argument.go b/accounts/abi/argument.go
index 9d97a2b4b9..b80d74289b 100644
--- a/accounts/abi/argument.go
+++ b/accounts/abi/argument.go
@@ -182,17 +182,12 @@ func (arguments Arguments) copyTuple(v interface{}, marshalledValues []interface
 // without supplying a struct to unpack into. Instead, this method returns a list containing the
 // values. An atomic argument will be a list with one element.
 func (arguments Arguments) UnpackValues(data []byte) ([]interface{}, error) {
-	size := 0
-	for _, arg := range arguments {
-		if arg.Indexed {
-			continue
-		}
-		size++
-	}
+	var (
+		retval      = make([]interface{}, 0)
+		virtualArgs = 0
+		index       = 0
+	)
 
-	retval := make([]interface{}, 0, size)
-	virtualArgs := 0
-	index := 0
 	for _, arg := range arguments {
 		if arg.Indexed {
 			continue