Skip to content

Commit b3be252

Browse files
brad4dcopybara-github
authored andcommitted
Use LinkedHash(Map|Set) to guarantee deterministic iteration order
PiperOrigin-RevId: 514566539
1 parent 6f388e1 commit b3be252

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/com/google/javascript/jscomp/ComposeWarningsGuard.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
import java.util.ArrayList;
2626
import java.util.Collections;
2727
import java.util.Comparator;
28-
import java.util.HashMap;
28+
import java.util.LinkedHashMap;
2929
import java.util.List;
3030
import java.util.Map;
3131
import java.util.SortedSet;
@@ -44,7 +44,7 @@ public final class ComposeWarningsGuard extends WarningsGuard {
4444
private static final long serialVersionUID = 1L;
4545

4646
// The order that the guards were added in.
47-
private final Map<WarningsGuard, Integer> orderOfAddition = new HashMap<>();
47+
private final Map<WarningsGuard, Integer> orderOfAddition = new LinkedHashMap<>();
4848
private int numberOfAdds = 0;
4949

5050
private final Comparator<WarningsGuard> guardComparator = new GuardComparator(orderOfAddition);

src/com/google/javascript/jscomp/DiagnosticGroup.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
import com.google.common.collect.ImmutableSet;
2020
import java.io.Serializable;
2121
import java.util.Arrays;
22-
import java.util.HashMap;
22+
import java.util.LinkedHashMap;
2323
import java.util.Map;
2424
import org.jspecify.nullness.Nullable;
2525

@@ -77,7 +77,7 @@ public DiagnosticGroup(String name, DiagnosticGroup... groups) {
7777
}
7878

7979
// DiagnosticGroups with only a single DiagnosticType.
80-
private static final Map<DiagnosticType, DiagnosticGroup> singletons = new HashMap<>();
80+
private static final Map<DiagnosticType, DiagnosticGroup> singletons = new LinkedHashMap<>();
8181

8282
/** Create a diagnostic group that matches only the given type. */
8383
public static synchronized DiagnosticGroup forType(DiagnosticType type) {

0 commit comments

Comments
 (0)