diff options
| author | Liam Miller-Cushon <cushon@google.com> | 2023-05-02 16:39:32 +0000 |
|---|---|---|
| committer | Sorin Basca <sorinbasca@google.com> | 2023-05-12 19:41:39 +0000 |
| commit | 1796f8c57714a2824830a8ddfdd1eab0f67de4eb (patch) | |
| tree | c6ece1557650c69315b6ca9283edcf75b1db0ee8 | |
| parent | c45dff0efc9a521edb97b5c3c14c104b80437e3a (diff) | |
| download | doclava-1796f8c57714a2824830a8ddfdd1eab0f67de4eb.tar.gz | |
Fix a concurrent modification exception when generating stubs
```
java.util.ConcurrentModificationException
at java.base/java.util.HashMap$HashIterator.nextNode(HashMap.java:1605)
at java.base/java.util.HashMap$ValueIterator.next(HashMap.java:1633)
at com.google.doclava.Stubs.writeStubsAndApi(Stubs.java:166)
at com.google.doclava.Doclava.start(Doclava.java:1731)
at com.google.doclava.Doclava.run(Doclava.java:1514)
at jdk.javadoc/jdk.javadoc.internal.tool.Start.parseAndExecute(Start.java:621)
at jdk.javadoc/jdk.javadoc.internal.tool.Start.begin(Start.java:444)
at jdk.javadoc/jdk.javadoc.internal.tool.Start.begin(Start.java:346)
at jdk.javadoc/jdk.javadoc.internal.tool.Main.execute(Main.java:57)
at jdk.javadoc/jdk.javadoc.internal.tool.Main.main(Main.java:46)
```
(cherry picked from https://android-review.googlesource.com/q/commit:e6e3b911ab40ad3ac0c45da6c8453ee72ef15aa7)
Merged-In: I7df9568d92764d8356cfe20001ff08b1b9d7e1ee
Change-Id: I7df9568d92764d8356cfe20001ff08b1b9d7e1ee
Bug: 278604691
| -rw-r--r-- | src/com/google/doclava/Stubs.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/com/google/doclava/Stubs.java b/src/com/google/doclava/Stubs.java index 930d52c..a90ed6e 100644 --- a/src/com/google/doclava/Stubs.java +++ b/src/com/google/doclava/Stubs.java @@ -53,7 +53,7 @@ public class Stubs { boolean keepStubComments) { // figure out which classes we need final HashSet<ClassInfo> notStrippable = new HashSet<ClassInfo>(); - Collection<ClassInfo> all = Converter.allClasses(); + Collection<ClassInfo> all = List.copyOf(Converter.allClasses()); Map<PackageInfo, List<ClassInfo>> allClassesByPackage = null; PrintStream apiWriter = null; PrintStream dexApiWriter = null; |