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 2020/07/09 06:26:17 UTC
[groovy] branch danielsun/tweak-java-stub updated: Trivial
refactoring: avoid redundant method calls
This is an automated email from the ASF dual-hosted git repository.
sunlan pushed a commit to branch danielsun/tweak-java-stub
in repository https://gitbox.apache.org/repos/asf/groovy.git
The following commit(s) were added to refs/heads/danielsun/tweak-java-stub by this push:
new 3960962 Trivial refactoring: avoid redundant method calls
3960962 is described below
commit 3960962ec498df4c46f0ca26c261e00223c7e271
Author: Daniel Sun <su...@apache.org>
AuthorDate: Thu Jul 9 14:25:58 2020 +0800
Trivial refactoring: avoid redundant method calls
---
src/main/java/org/codehaus/groovy/tools/javac/JavaStubGenerator.java | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/main/java/org/codehaus/groovy/tools/javac/JavaStubGenerator.java b/src/main/java/org/codehaus/groovy/tools/javac/JavaStubGenerator.java
index ead34a9..fe9addd 100644
--- a/src/main/java/org/codehaus/groovy/tools/javac/JavaStubGenerator.java
+++ b/src/main/java/org/codehaus/groovy/tools/javac/JavaStubGenerator.java
@@ -459,8 +459,9 @@ public class JavaStubGenerator {
boolean isInterface = isInterfaceOrTrait(classNode);
List<FieldNode> fields = classNode.getFields();
if (fields == null) return;
- List<FieldNode> enumFields = new ArrayList<FieldNode>(fields.size());
- List<FieldNode> normalFields = new ArrayList<FieldNode>(fields.size());
+ final int fieldCnt = fields.size();
+ List<FieldNode> enumFields = new ArrayList<FieldNode>(fieldCnt);
+ List<FieldNode> normalFields = new ArrayList<FieldNode>(fieldCnt);
for (FieldNode field : fields) {
boolean isSynthetic = (field.getModifiers() & Opcodes.ACC_SYNTHETIC) != 0;
if (field.isEnum()) {