You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2022/12/14 15:53:36 UTC

[groovy] branch master updated: Trivial refactoring: specify the size of list

This is an automated email from the ASF dual-hosted git repository.

sunlan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/master by this push:
     new 191e4fb9b5 Trivial refactoring: specify the size of list
191e4fb9b5 is described below

commit 191e4fb9b501231fc856feee5e80a93d17a199a6
Author: Daniel Sun <su...@apache.org>
AuthorDate: Wed Dec 14 23:52:34 2022 +0800

    Trivial refactoring: specify the size of list
---
 .../java/org/codehaus/groovy/classgen/AsmClassGenerator.java   | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/main/java/org/codehaus/groovy/classgen/AsmClassGenerator.java b/src/main/java/org/codehaus/groovy/classgen/AsmClassGenerator.java
index b209df0b1e..8dc6932706 100644
--- a/src/main/java/org/codehaus/groovy/classgen/AsmClassGenerator.java
+++ b/src/main/java/org/codehaus/groovy/classgen/AsmClassGenerator.java
@@ -2299,10 +2299,12 @@ public class AsmClassGenerator extends ClassGenerator {
     }
 
     public void despreadList(final List<Expression> expressions, final boolean wrap) {
-        List<Expression> spreadIndexes = new ArrayList<>();
-        List<Expression> spreadExpressions = new ArrayList<>();
-        List<Expression> normalArguments = new ArrayList<>();
-        for (int i = 0, n = expressions.size(); i < n; i += 1) {
+        final int expressionCnt = expressions.size();
+        List<Expression> spreadIndexes = new ArrayList<>(expressionCnt);
+        List<Expression> spreadExpressions = new ArrayList<>(expressionCnt);
+        List<Expression> normalArguments = new ArrayList<>(expressionCnt);
+
+        for (int i = 0; i < expressionCnt; i += 1) {
             Expression expr = expressions.get(i);
             if (!(expr instanceof SpreadExpression)) {
                 normalArguments.add(expr);