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);