This is an automated email from the ASF dual-hosted git repository.
paulk 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 93d45f230c minor refactor: keep some of the groovydoc map data
structures ordered for greater repeatability
93d45f230c is described below
commit 93d45f230c9c3fe3f243ac6d86bea796088ede84
Author: Paul King <[email protected]>
AuthorDate: Thu May 1 20:37:29 2025 +1000
minor refactor: keep some of the groovydoc map data structures ordered for
greater repeatability
---
.../src/main/java/org/apache/groovy/antlr/GroovydocVisitor.java | 6 +++---
.../org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java | 3 +--
.../org/codehaus/groovy/tools/groovydoc/SimpleGroovyRootDoc.java | 7 +++----
.../groovy/tools/groovydoc/antlr4/GroovydocJavaVisitor.java | 6 +++---
4 files changed, 10 insertions(+), 12 deletions(-)
diff --git
a/subprojects/groovy-groovydoc/src/main/java/org/apache/groovy/antlr/GroovydocVisitor.java
b/subprojects/groovy-groovydoc/src/main/java/org/apache/groovy/antlr/GroovydocVisitor.java
index 0f27cabfae..51690fa3ae 100644
---
a/subprojects/groovy-groovydoc/src/main/java/org/apache/groovy/antlr/GroovydocVisitor.java
+++
b/subprojects/groovy-groovydoc/src/main/java/org/apache/groovy/antlr/GroovydocVisitor.java
@@ -53,8 +53,8 @@ import org.codehaus.groovy.tools.groovydoc.SimpleGroovyType;
import java.lang.reflect.Modifier;
import java.util.ArrayList;
-import java.util.HashMap;
import java.util.Iterator;
+import java.util.LinkedHashMap;
import java.util.List;
import java.util.Map;
import java.util.Properties;
@@ -74,7 +74,7 @@ public class GroovydocVisitor extends ClassCodeVisitorSupport
{
private final List<LinkArgument> links;
private String packagePath;
private SimpleGroovyClassDoc currentClassDoc = null;
- private Map<String, GroovyClassDoc> classDocs = new HashMap<>();
+ private Map<String, GroovyClassDoc> classDocs = new LinkedHashMap<>();
private final Properties properties;
private static final String FS = "/";
@@ -96,7 +96,7 @@ public class GroovydocVisitor extends ClassCodeVisitorSupport
{
@Override
public void visitClass(ClassNode node) {
- final Map<String, String> aliases = new HashMap<>();
+ final Map<String, String> aliases = new LinkedHashMap<>();
final List<String> imports = new ArrayList<>();
for (ImportNode iNode : node.getModule().getImports()) {
String name = iNode.getClassName();
diff --git
a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
index ba470216f4..f6fa781e7e 100644
---
a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
+++
b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
@@ -34,7 +34,6 @@ import org.codehaus.groovy.runtime.DefaultGroovyMethods;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collections;
-import java.util.HashMap;
import java.util.LinkedHashMap;
import java.util.LinkedHashSet;
import java.util.LinkedList;
@@ -113,7 +112,7 @@ public class SimpleGroovyClassDoc extends
SimpleGroovyAbstractableElementDoc imp
interfaceNames = new ArrayList<>();
interfaceClasses = new ArrayList<>();
nested = new ArrayList<>();
- resolvedExternalClassesCache = new HashMap<>();
+ resolvedExternalClassesCache = new LinkedHashMap<>();
}
public SimpleGroovyClassDoc(List<String> importedClassesAndPackages,
Map<String, String> aliases, String name) {
diff --git
a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyRootDoc.java
b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyRootDoc.java
index c7b1dfef5f..931e910ec4 100644
---
a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyRootDoc.java
+++
b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyRootDoc.java
@@ -24,7 +24,6 @@ import org.codehaus.groovy.groovydoc.GroovyRootDoc;
import java.util.ArrayList;
import java.util.Collections;
-import java.util.HashMap;
import java.util.LinkedHashMap;
import java.util.List;
import java.util.Map;
@@ -41,7 +40,7 @@ public class SimpleGroovyRootDoc extends SimpleGroovyDoc
implements GroovyRootDo
private final Map<String, GroovyClassDoc> classDocs;
private final Map<String, String> equivalentPackageImports;
private List<GroovyClassDoc> classDocValues = null;
- private final Map<String, GroovyClassDoc> cachedResolvedClasses = new
HashMap<>();
+ private final Map<String, GroovyClassDoc> cachedResolvedClasses = new
LinkedHashMap<>();
private final ClassNamedCache classNamedCache;
private String description = "";
@@ -50,7 +49,7 @@ public class SimpleGroovyRootDoc extends SimpleGroovyDoc
implements GroovyRootDo
super(name);
packageDocs = new LinkedHashMap<>();
classDocs = new LinkedHashMap<>();
- equivalentPackageImports = new HashMap<>();
+ equivalentPackageImports = new LinkedHashMap<>();
classNamedCache = new ClassNamedCache(classDocs);
}
@@ -175,7 +174,7 @@ public class SimpleGroovyRootDoc extends SimpleGroovyDoc
implements GroovyRootDo
private static final class ClassNamedCache {
private final Map<String, GroovyClassDoc> classDocs;
- private final Map<Entry, GroovyClassDoc> store = new HashMap<>();
+ private final Map<Entry, GroovyClassDoc> store = new LinkedHashMap<>();
private ClassNamedCache(final Map<String, GroovyClassDoc> classDocs) {
this.classDocs = classDocs;
diff --git
a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/antlr4/GroovydocJavaVisitor.java
b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/antlr4/GroovydocJavaVisitor.java
index 22bc285adc..dc40233634 100644
---
a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/antlr4/GroovydocJavaVisitor.java
+++
b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/antlr4/GroovydocJavaVisitor.java
@@ -57,7 +57,7 @@ import
org.codehaus.groovy.tools.groovydoc.SimpleGroovyProgramElementDoc;
import org.codehaus.groovy.tools.groovydoc.SimpleGroovyType;
import java.util.ArrayList;
-import java.util.HashMap;
+import java.util.LinkedHashMap;
import java.util.List;
import java.util.Map;
import java.util.Optional;
@@ -65,9 +65,9 @@ import java.util.Optional;
public class GroovydocJavaVisitor extends VoidVisitorAdapter<Object> {
private final List<LinkArgument> links;
private SimpleGroovyClassDoc currentClassDoc = null;
- private Map<String, GroovyClassDoc> classDocs = new HashMap<>();
+ private Map<String, GroovyClassDoc> classDocs = new LinkedHashMap<>();
private String packagePath;
- private final Map<String, String> aliases = new HashMap<>();
+ private final Map<String, String> aliases = new LinkedHashMap<>();
private List<String> imports = new ArrayList<>();
private static final String FS = "/";
