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
commit 7db710ba3b42d4282888c197bcf8c7633eab3cd8 Author: Daniel Sun <[email protected]> AuthorDate: Sat Mar 29 18:53:29 2025 +0900 Trivial refactoring: eliminate redundant `Collection` operation (cherry picked from commit c157dd64ece7767f4a2740beeb30eeaa2f7473a7) --- src/main/java/org/codehaus/groovy/runtime/InvokerHelper.java | 2 +- .../groovy/tools/groovydoc/SimpleGroovyPackageDoc.java | 10 +++++----- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/main/java/org/codehaus/groovy/runtime/InvokerHelper.java b/src/main/java/org/codehaus/groovy/runtime/InvokerHelper.java index 7e4f547e53..bc4a512a2b 100644 --- a/src/main/java/org/codehaus/groovy/runtime/InvokerHelper.java +++ b/src/main/java/org/codehaus/groovy/runtime/InvokerHelper.java @@ -350,7 +350,7 @@ public class InvokerHelper { public static SpreadMap spreadMap(Object value) { if (value instanceof Map) { - Object[] values = new Object[((Map) value).keySet().size() * 2]; + Object[] values = new Object[((Map) value).size() * 2]; int index = 0; for (Object key : ((Map) value).keySet()) { values[index++] = key; diff --git a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyPackageDoc.java b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyPackageDoc.java index 340edb4349..594bda52eb 100644 --- a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyPackageDoc.java +++ b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyPackageDoc.java @@ -77,7 +77,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac @Override public GroovyClassDoc[] enums() { - List<GroovyClassDoc> result = new ArrayList<GroovyClassDoc>(classDocs.values().size()); + List<GroovyClassDoc> result = new ArrayList<GroovyClassDoc>(classDocs.size()); for (GroovyClassDoc doc : classDocs.values()) { if (doc.isEnum()) { result.add(doc); @@ -88,7 +88,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac @Override public GroovyClassDoc[] errors() { - List<GroovyClassDoc> result = new ArrayList<GroovyClassDoc>(classDocs.values().size()); + List<GroovyClassDoc> result = new ArrayList<GroovyClassDoc>(classDocs.size()); for (GroovyClassDoc doc : classDocs.values()) { if (doc.isError()) { result.add(doc); @@ -99,7 +99,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac @Override public GroovyClassDoc[] exceptions() { - List<GroovyClassDoc> result = new ArrayList<GroovyClassDoc>(classDocs.values().size()); + List<GroovyClassDoc> result = new ArrayList<GroovyClassDoc>(classDocs.size()); for (GroovyClassDoc doc : classDocs.values()) { if (doc.isException()) { result.add(doc); @@ -115,7 +115,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac @Override public GroovyClassDoc[] interfaces() { - List<GroovyClassDoc> result = new ArrayList<GroovyClassDoc>(classDocs.values().size()); + List<GroovyClassDoc> result = new ArrayList<GroovyClassDoc>(classDocs.size()); for (GroovyClassDoc doc : classDocs.values()) { if (doc.isInterface()) { result.add(doc); @@ -126,7 +126,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac @Override public GroovyClassDoc[] ordinaryClasses() { - List<GroovyClassDoc> result = new ArrayList<GroovyClassDoc>(classDocs.values().size()); + List<GroovyClassDoc> result = new ArrayList<GroovyClassDoc>(classDocs.size()); for (GroovyClassDoc doc : classDocs.values()) { if (doc.isOrdinaryClass()) { result.add(doc);
