Skip to content

Commit 8bbbefd

Browse files
cpovirkGoogle Java Core Libraries
authored andcommitted
Use _more_ static imports.
(followup to cl/687404334) RELNOTES=n/a PiperOrigin-RevId: 687466612
1 parent 266b30c commit 8bbbefd

File tree

290 files changed

+1619
-1536
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

290 files changed

+1619
-1536
lines changed

android/guava-testlib/src/com/google/common/collect/testing/AbstractContainerTester.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,12 @@
1616

1717
package com.google.common.collect.testing;
1818

19+
import static java.util.Arrays.asList;
20+
1921
import com.google.common.annotations.GwtCompatible;
2022
import com.google.errorprone.annotations.CanIgnoreReturnValue;
2123
import com.google.errorprone.annotations.OverridingMethodsMustInvokeSuper;
2224
import java.util.ArrayList;
23-
import java.util.Arrays;
2425
import java.util.Collection;
2526
import java.util.Collections;
2627
import java.util.List;
@@ -91,7 +92,7 @@ protected C resetContainer(C newValue) {
9192
* @param elements expected contents of {@link #container}
9293
*/
9394
protected final void expectContents(E... elements) {
94-
expectContents(Arrays.asList(elements));
95+
expectContents(asList(elements));
9596
}
9697

9798
/**
@@ -139,12 +140,12 @@ protected void expectUnchanged() {
139140
*/
140141
protected final void expectAdded(E... elements) {
141142
List<E> expected = Helpers.copyToList(getSampleElements());
142-
expected.addAll(Arrays.asList(elements));
143+
expected.addAll(asList(elements));
143144
expectContents(expected);
144145
}
145146

146147
protected final void expectAdded(int index, E... elements) {
147-
expectAdded(index, Arrays.asList(elements));
148+
expectAdded(index, asList(elements));
148149
}
149150

150151
protected final void expectAdded(int index, Collection<E> elements) {

android/guava-testlib/src/com/google/common/collect/testing/AbstractIteratorTester.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616

1717
package com.google.common.collect.testing;
1818

19+
import static java.util.Arrays.asList;
1920
import static junit.framework.Assert.assertEquals;
2021
import static junit.framework.Assert.fail;
2122

@@ -340,7 +341,7 @@ private void recurse(int level) {
340341
}
341342

342343
private void compareResultsForThisListOfStimuli() {
343-
int removes = Collections.frequency(Arrays.asList(stimuli), remove);
344+
int removes = Collections.frequency(asList(stimuli), remove);
344345
if ((!features.contains(IteratorFeature.SUPPORTS_REMOVE) && removes > 1)
345346
|| (stimuli.length >= 5 && removes > 2)) {
346347
// removes are the most expensive thing to test, since they often throw exceptions with stack
@@ -363,7 +364,7 @@ private void compareResultsForThisListOfStimuli() {
363364
private static List<Object> subListCopy(Object[] source, int size) {
364365
final Object[] copy = new Object[size];
365366
System.arraycopy(source, 0, copy, 0, size);
366-
return Arrays.asList(copy);
367+
return asList(copy);
367368
}
368369

369370
private interface IteratorOperation {
@@ -539,7 +540,7 @@ void executeAndCompare(ListIterator<E> reference, Iterator<E> target) {
539540
};
540541

541542
List<Stimulus<E, Iterator<E>>> iteratorStimuli() {
542-
return Arrays.asList(hasNext, next, remove);
543+
return asList(hasNext, next, remove);
543544
}
544545

545546
Stimulus<E, ListIterator<E>> hasPrevious =
@@ -586,6 +587,6 @@ void executeAndCompare(ListIterator<E> reference, ListIterator<E> target) {
586587
};
587588

588589
List<Stimulus<E, ListIterator<E>>> listIteratorStimuli() {
589-
return Arrays.asList(hasPrevious, nextIndex, previousIndex, previous, add, set);
590+
return asList(hasPrevious, nextIndex, previousIndex, previous, add, set);
590591
}
591592
}

android/guava-testlib/src/com/google/common/collect/testing/ConcurrentMapTestSuiteBuilder.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,13 @@
1414

1515
package com.google.common.collect.testing;
1616

17+
import static java.util.Arrays.asList;
18+
1719
import com.google.common.annotations.GwtIncompatible;
1820
import com.google.common.collect.testing.testers.ConcurrentMapPutIfAbsentTester;
1921
import com.google.common.collect.testing.testers.ConcurrentMapRemoveTester;
2022
import com.google.common.collect.testing.testers.ConcurrentMapReplaceEntryTester;
2123
import com.google.common.collect.testing.testers.ConcurrentMapReplaceTester;
22-
import java.util.Arrays;
2324
import java.util.List;
2425

2526
/**
@@ -38,7 +39,7 @@ public static <K, V> ConcurrentMapTestSuiteBuilder<K, V> using(TestMapGenerator<
3839

3940
@SuppressWarnings("rawtypes") // class literals
4041
static final List<? extends Class<? extends AbstractTester>> TESTERS =
41-
Arrays.asList(
42+
asList(
4243
ConcurrentMapPutIfAbsentTester.class,
4344
ConcurrentMapRemoveTester.class,
4445
ConcurrentMapReplaceTester.class,

android/guava-testlib/src/com/google/common/collect/testing/DerivedCollectionGenerators.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,11 @@
1919
import static com.google.common.collect.testing.Helpers.castOrCopyToList;
2020
import static com.google.common.collect.testing.Helpers.equal;
2121
import static com.google.common.collect.testing.Helpers.mapEntry;
22+
import static java.util.Arrays.asList;
2223
import static java.util.Collections.sort;
2324

2425
import com.google.common.annotations.GwtCompatible;
2526
import java.util.ArrayList;
26-
import java.util.Arrays;
2727
import java.util.Collection;
2828
import java.util.Collections;
2929
import java.util.Comparator;
@@ -382,7 +382,7 @@ public Iterable<E> order(List<E> insertionOrder) {
382382

383383
@Override
384384
public SortedSet<E> create(Object... elements) {
385-
List<?> normalValues = (List<?>) Arrays.asList(elements);
385+
List<?> normalValues = (List<?>) asList(elements);
386386
List<E> extremeValues = new ArrayList<>();
387387

388388
// nulls are usually out of bounds for a subset, so ban them altogether
@@ -472,7 +472,7 @@ public SortedMapSubmapTestMapGenerator(
472472
// derive values for inclusive filtering from the input samples
473473
SampleElements<Entry<K, V>> samples = delegate.samples();
474474
List<Entry<K, V>> samplesList =
475-
Arrays.asList(samples.e0(), samples.e1(), samples.e2(), samples.e3(), samples.e4());
475+
asList(samples.e0(), samples.e1(), samples.e2(), samples.e3(), samples.e4());
476476
Collections.sort(samplesList, entryComparator);
477477
this.firstInclusive = samplesList.get(0).getKey();
478478
this.lastInclusive = samplesList.get(samplesList.size() - 1).getKey();

android/guava-testlib/src/com/google/common/collect/testing/FeatureSpecificTestSuiteBuilder.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616

1717
package com.google.common.collect.testing;
1818

19+
import static java.util.Arrays.asList;
1920
import static java.util.Collections.disjoint;
2021
import static java.util.logging.Level.FINER;
2122

@@ -27,7 +28,6 @@
2728
import com.google.errorprone.annotations.CanIgnoreReturnValue;
2829
import java.lang.reflect.Method;
2930
import java.util.ArrayList;
30-
import java.util.Arrays;
3131
import java.util.Collection;
3232
import java.util.Collections;
3333
import java.util.Enumeration;
@@ -109,7 +109,7 @@ public Runnable getTearDown() {
109109
*/
110110
@CanIgnoreReturnValue
111111
public B withFeatures(Feature<?>... features) {
112-
return withFeatures(Arrays.asList(features));
112+
return withFeatures(asList(features));
113113
}
114114

115115
@CanIgnoreReturnValue
@@ -157,7 +157,7 @@ public String getName() {
157157
*/
158158
@CanIgnoreReturnValue
159159
public B suppressing(Method... methods) {
160-
return suppressing(Arrays.asList(methods));
160+
return suppressing(asList(methods));
161161
}
162162

163163
@CanIgnoreReturnValue

android/guava-testlib/src/com/google/common/collect/testing/Helpers.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
package com.google.common.collect.testing;
1818

1919
import static java.lang.Math.max;
20+
import static java.util.Arrays.asList;
2021
import static java.util.Collections.sort;
2122
import static junit.framework.Assert.assertEquals;
2223
import static junit.framework.Assert.assertFalse;
@@ -31,7 +32,6 @@
3132
import java.lang.reflect.Method;
3233
import java.util.AbstractList;
3334
import java.util.ArrayList;
34-
import java.util.Arrays;
3535
import java.util.Collection;
3636
import java.util.Collections;
3737
import java.util.Comparator;
@@ -60,7 +60,7 @@ static boolean equal(@Nullable Object a, @Nullable Object b) {
6060
}
6161

6262
public static <E extends @Nullable Object> List<E> copyToList(E[] elements) {
63-
return copyToList(Arrays.asList(elements));
63+
return copyToList(asList(elements));
6464
}
6565

6666
// Clone of Sets.newLinkedHashSet
@@ -71,7 +71,7 @@ static boolean equal(@Nullable Object a, @Nullable Object b) {
7171
}
7272

7373
public static <E extends @Nullable Object> Set<E> copyToSet(E[] elements) {
74-
return copyToSet(Arrays.asList(elements));
74+
return copyToSet(asList(elements));
7575
}
7676

7777
// Would use Maps.immutableEntry
@@ -125,7 +125,7 @@ public static void assertEqualInOrder(Iterable<?> expected, Iterable<?> actual)
125125
}
126126

127127
public static void assertContentsInOrder(Iterable<?> actual, Object... expected) {
128-
assertEqualInOrder(Arrays.asList(expected), actual);
128+
assertEqualInOrder(asList(expected), actual);
129129
}
130130

131131
public static void assertEqualIgnoringOrder(Iterable<?> expected, Iterable<?> actual) {
@@ -153,7 +153,7 @@ public static void assertEqualIgnoringOrder(Iterable<?> expected, Iterable<?> ac
153153
}
154154

155155
public static void assertContentsAnyOrder(Iterable<?> actual, Object... expected) {
156-
assertEqualIgnoringOrder(Arrays.asList(expected), actual);
156+
assertEqualIgnoringOrder(asList(expected), actual);
157157
}
158158

159159
public static void assertContains(Iterable<?> actual, Object expected) {
@@ -175,14 +175,14 @@ public static void assertContains(Iterable<?> actual, Object expected) {
175175
}
176176

177177
public static void assertContainsAllOf(Iterable<?> actual, Object... expected) {
178-
List<Object> expectedList = new ArrayList<>(Arrays.asList(expected));
178+
List<Object> expectedList = new ArrayList<>(asList(expected));
179179

180180
for (Object o : actual) {
181181
expectedList.remove(o);
182182
}
183183

184184
if (!expectedList.isEmpty()) {
185-
fail("Not true that " + actual + " contains all of " + Arrays.asList(expected));
185+
fail("Not true that " + actual + " contains all of " + asList(expected));
186186
}
187187
}
188188

@@ -283,7 +283,7 @@ Comparator<Entry<K, V>> entryComparator(@Nullable Comparator<? super K> keyCompa
283283
*/
284284
public static <T extends @Nullable Object> void testComparator(
285285
Comparator<? super T> comparator, T... valuesInExpectedOrder) {
286-
testComparator(comparator, Arrays.asList(valuesInExpectedOrder));
286+
testComparator(comparator, asList(valuesInExpectedOrder));
287287
}
288288

289289
/**

android/guava-testlib/src/com/google/common/collect/testing/MapInterfaceTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,11 @@
1717
package com.google.common.collect.testing;
1818

1919
import static com.google.common.collect.testing.ReflectionFreeAssertThrows.assertThrows;
20+
import static java.util.Arrays.asList;
2021
import static java.util.Collections.singleton;
2122

2223
import com.google.common.annotations.GwtCompatible;
2324
import com.google.common.annotations.J2ktIncompatible;
24-
import java.util.Arrays;
2525
import java.util.Collection;
2626
import java.util.Collections;
2727
import java.util.HashSet;
@@ -226,23 +226,23 @@ protected final void assertInvariants(Map<K, V> map) {
226226

227227
Object[] entrySetToArray1 = entrySet.toArray();
228228
assertEquals(map.size(), entrySetToArray1.length);
229-
assertTrue(Arrays.asList(entrySetToArray1).containsAll(entrySet));
229+
assertTrue(asList(entrySetToArray1).containsAll(entrySet));
230230

231231
Entry<?, ?>[] entrySetToArray2 = new Entry<?, ?>[map.size() + 2];
232232
entrySetToArray2[map.size()] = mapEntry("foo", 1);
233233
assertSame(entrySetToArray2, entrySet.toArray(entrySetToArray2));
234234
assertNull(entrySetToArray2[map.size()]);
235-
assertTrue(Arrays.asList(entrySetToArray2).containsAll(entrySet));
235+
assertTrue(asList(entrySetToArray2).containsAll(entrySet));
236236

237237
Object[] valuesToArray1 = valueCollection.toArray();
238238
assertEquals(map.size(), valuesToArray1.length);
239-
assertTrue(Arrays.asList(valuesToArray1).containsAll(valueCollection));
239+
assertTrue(asList(valuesToArray1).containsAll(valueCollection));
240240

241241
Object[] valuesToArray2 = new Object[map.size() + 2];
242242
valuesToArray2[map.size()] = "foo";
243243
assertSame(valuesToArray2, valueCollection.toArray(valuesToArray2));
244244
assertNull(valuesToArray2[map.size()]);
245-
assertTrue(Arrays.asList(valuesToArray2).containsAll(valueCollection));
245+
assertTrue(asList(valuesToArray2).containsAll(valueCollection));
246246

247247
if (supportsValuesHashCode) {
248248
int expectedHash = 0;

android/guava-testlib/src/com/google/common/collect/testing/MinimalCollection.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,10 @@
1616

1717
package com.google.common.collect.testing;
1818

19+
import static java.util.Arrays.asList;
20+
1921
import com.google.common.annotations.GwtCompatible;
2022
import java.util.AbstractCollection;
21-
import java.util.Arrays;
2223
import java.util.Collection;
2324
import java.util.Iterator;
2425
import org.checkerframework.checker.nullness.qual.NonNull;
@@ -79,7 +80,7 @@ public boolean contains(@Nullable Object object) {
7980
}
8081
}
8182
Platform.checkCast(type, object); // behave badly
82-
return Arrays.asList(contents).contains(object);
83+
return asList(contents).contains(object);
8384
}
8485

8586
@Override
@@ -97,7 +98,7 @@ public boolean containsAll(Collection<?> collection) {
9798

9899
@Override
99100
public Iterator<E> iterator() {
100-
return Arrays.asList(contents).iterator();
101+
return asList(contents).iterator();
101102
}
102103

103104
@Override

android/guava-testlib/src/com/google/common/collect/testing/MinimalIterable.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,9 @@
1616

1717
package com.google.common.collect.testing;
1818

19+
import static java.util.Arrays.asList;
20+
1921
import com.google.common.annotations.GwtCompatible;
20-
import java.util.Arrays;
2122
import java.util.Collection;
2223
import java.util.Iterator;
2324
import org.checkerframework.checker.nullness.qual.Nullable;
@@ -52,7 +53,7 @@ public final class MinimalIterable<E extends @Nullable Object> implements Iterab
5253
/** Returns an iterable whose iterator returns the given elements in order. */
5354
public static <E extends @Nullable Object> MinimalIterable<E> of(E... elements) {
5455
// Make sure to get an unmodifiable iterator
55-
return new MinimalIterable<>(Arrays.asList(elements).iterator());
56+
return new MinimalIterable<>(asList(elements).iterator());
5657
}
5758

5859
/**

android/guava-testlib/src/com/google/common/collect/testing/MinimalSet.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,10 @@
1616

1717
package com.google.common.collect.testing;
1818

19+
import static java.util.Arrays.asList;
20+
1921
import com.google.common.annotations.GwtCompatible;
2022
import java.util.ArrayList;
21-
import java.util.Arrays;
2223
import java.util.Collection;
2324
import java.util.List;
2425
import java.util.Set;
@@ -38,7 +39,7 @@ public class MinimalSet<E extends @Nullable Object> extends MinimalCollection<E>
3839

3940
@SuppressWarnings("unchecked") // empty Object[] as E[]
4041
public static <E extends @Nullable Object> MinimalSet<E> of(E... contents) {
41-
return ofClassAndContents(Object.class, (E[]) new Object[0], Arrays.asList(contents));
42+
return ofClassAndContents(Object.class, (E[]) new Object[0], asList(contents));
4243
}
4344

4445
@SuppressWarnings("unchecked") // empty Object[] as E[]

0 commit comments

Comments
 (0)