Statically import some methods from Collections

PicnicSupermarket · Apr 12, 2022 · 0f6549a · 0f6549a
1 parent 862ab44
commit 0f6549a
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 13 deletions.
diff --git a/...one-contrib/src/main/java/tech/picnic/errorprone/refastertemplates/AssortedTemplates.java b/...one-contrib/src/main/java/tech/picnic/errorprone/refastertemplates/AssortedTemplates.java
@@ -4,6 +4,7 @@
 import static com.google.common.collect.ImmutableSet.toImmutableSet;
 import static com.google.common.collect.Sets.toImmutableEnumSet;
 import static com.google.errorprone.refaster.ImportPolicy.STATIC_IMPORT_ALWAYS;
+import static java.util.Collections.disjoint;
 import static java.util.Objects.checkIndex;
 
 import com.google.common.base.Splitter;
@@ -162,7 +163,7 @@ boolean before2(Set<T> set1, Set<T> set2) {
 
     @AfterTemplate
     boolean after(Set<T> set1, Set<T> set2) {
-      return Collections.disjoint(set1, set2);
+      return disjoint(set1, set2);
     }
   }
 
@@ -177,15 +178,15 @@ static final class DisjointCollections<T> {
     @BeforeTemplate
     boolean before(Collection<T> collection1, Collection<T> collection2) {
       return Refaster.anyOf(
-          Collections.disjoint(ImmutableSet.copyOf(collection1), collection2),
-          Collections.disjoint(new HashSet<>(collection1), collection2),
-          Collections.disjoint(collection1, ImmutableSet.copyOf(collection2)),
-          Collections.disjoint(collection1, new HashSet<>(collection2)));
+          disjoint(ImmutableSet.copyOf(collection1), collection2),
+          disjoint(new HashSet<>(collection1), collection2),
+          disjoint(collection1, ImmutableSet.copyOf(collection2)),
+          disjoint(collection1, new HashSet<>(collection2)));
     }
 
     @AfterTemplate
     boolean after(Collection<T> collection1, Collection<T> collection2) {
-      return Collections.disjoint(collection1, collection2);
+      return disjoint(collection1, collection2);
     }
   }
 

diff --git a/...ontrib/src/main/java/tech/picnic/errorprone/refastertemplates/ImmutableListTemplates.java b/...ontrib/src/main/java/tech/picnic/errorprone/refastertemplates/ImmutableListTemplates.java
@@ -3,6 +3,7 @@
 import static com.google.common.collect.ImmutableList.toImmutableList;
 import static com.google.common.collect.ImmutableSet.toImmutableSet;
 import static com.google.errorprone.refaster.ImportPolicy.STATIC_IMPORT_ALWAYS;
+import static java.util.Collections.singletonList;
 import static java.util.Comparator.naturalOrder;
 import static java.util.stream.Collectors.collectingAndThen;
 import static java.util.stream.Collectors.toList;
@@ -16,7 +17,6 @@
 import com.google.errorprone.refaster.annotation.UseImportPolicy;
 import java.util.Arrays;
 import java.util.Collection;
-import java.util.Collections;
 import java.util.Comparator;
 import java.util.Iterator;
 import java.util.List;
@@ -63,7 +63,7 @@ ImmutableList<T> after() {
   static final class SingletonImmutableList<T> {
     @BeforeTemplate
     List<T> before(T element) {
-      return Collections.singletonList(element);
+      return singletonList(element);
     }
 
     @AfterTemplate

diff --git a/...contrib/src/main/java/tech/picnic/errorprone/refastertemplates/ImmutableMapTemplates.java b/...contrib/src/main/java/tech/picnic/errorprone/refastertemplates/ImmutableMapTemplates.java
@@ -2,6 +2,7 @@
 
 import static com.google.common.collect.ImmutableMap.toImmutableMap;
 import static com.google.errorprone.refaster.ImportPolicy.STATIC_IMPORT_ALWAYS;
+import static java.util.Collections.singletonMap;
 import static java.util.function.Function.identity;
 
 import com.google.common.collect.ImmutableMap;
@@ -14,7 +15,6 @@
 import com.google.errorprone.refaster.annotation.Placeholder;
 import com.google.errorprone.refaster.annotation.UseImportPolicy;
 import java.util.Collection;
-import java.util.Collections;
 import java.util.Iterator;
 import java.util.Map;
 import java.util.Set;
@@ -65,8 +65,7 @@ static final class PairToImmutableMap<K, V> {
     @BeforeTemplate
     Map<K, V> before(K key, V value) {
       return Refaster.anyOf(
-          ImmutableMap.<K, V>builder().put(key, value).build(),
-          Collections.singletonMap(key, value));
+          ImmutableMap.<K, V>builder().put(key, value).build(), singletonMap(key, value));
     }
 
     @AfterTemplate

diff --git a/...contrib/src/main/java/tech/picnic/errorprone/refastertemplates/ImmutableSetTemplates.java b/...contrib/src/main/java/tech/picnic/errorprone/refastertemplates/ImmutableSetTemplates.java
@@ -2,6 +2,7 @@
 
 import static com.google.common.collect.ImmutableSet.toImmutableSet;
 import static com.google.errorprone.refaster.ImportPolicy.STATIC_IMPORT_ALWAYS;
+import static java.util.Collections.singleton;
 import static java.util.stream.Collectors.collectingAndThen;
 import static java.util.stream.Collectors.toList;
 import static java.util.stream.Collectors.toSet;
@@ -15,7 +16,6 @@
 import com.google.errorprone.refaster.annotation.UseImportPolicy;
 import java.util.Arrays;
 import java.util.Collection;
-import java.util.Collections;
 import java.util.Iterator;
 import java.util.Set;
 import java.util.stream.Stream;
@@ -61,7 +61,7 @@ ImmutableSet<T> after() {
   static final class SingletonImmutableSet<T> {
     @BeforeTemplate
     Set<T> before(T element) {
-      return Collections.singleton(element);
+      return singleton(element);
     }
 
     @AfterTemplate