(groovy) branch master updated: Trivial refactoring: eliminate redundant `Collection` operation

[sunlan]

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 c157dd64ec Trivial refactoring: eliminate redundant `Collection` 
operation
c157dd64ec is described below

commit c157dd64ece7767f4a2740beeb30eeaa2f7473a7
Author: Daniel Sun <sun...@apache.org>
AuthorDate: Sat Mar 29 18:53:29 2025 +0900

    Trivial refactoring: eliminate redundant `Collection` operation
---
 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 c05b00abec..847542da48 100644
--- a/src/main/java/org/codehaus/groovy/runtime/InvokerHelper.java
+++ b/src/main/java/org/codehaus/groovy/runtime/InvokerHelper.java
@@ -348,7 +348,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);