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 16:18:06 UTC
[groovy] branch GROOVY_4_0_X 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 GROOVY_4_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git
The following commit(s) were added to refs/heads/GROOVY_4_0_X by this push:
new 05641b9f95 Trivial refactoring: specify the size of list
05641b9f95 is described below
commit 05641b9f9553e699f797eaad6eade6e30692ff08
Author: Daniel Sun <su...@apache.org>
AuthorDate: Wed Dec 14 23:52:34 2022 +0800
Trivial refactoring: specify the size of list
(cherry picked from commit 191e4fb9b501231fc856feee5e80a93d17a199a6)
---
.../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 0f2af490a4..32993f3341 100644
--- a/src/main/java/org/codehaus/groovy/classgen/AsmClassGenerator.java
+++ b/src/main/java/org/codehaus/groovy/classgen/AsmClassGenerator.java
@@ -2294,10 +2294,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);