ImmutableSortedSetRules

SUGGESTION

Simplification

View source code on GitHub

Suppression

Disable all rules by adding -XepOpt:Refaster:NamePattern=^(?!ImmutableSortedSetRules\$).* as compiler argument.

Table of contents

1. ImmutableSortedSetOrderedBy
2. ImmutableSortedSetNaturalOrder
3. ImmutableSortedSetReverseOrder
4. ImmutableSortedSetOf
5. ImmutableSortedSetCopyOf
6. StreamCollectToImmutableSortedSetNaturalOrder

ImmutableSortedSetOrderedBy

SUGGESTION

Simplification

Suppression

Suppress false positives by adding the suppression annotation @SuppressWarnings("ImmutableSortedSetOrderedBy") to the enclosing element.

Disable this rule by adding -XepOpt:Refaster:NamePattern=^(?!ImmutableSortedSetRules\$ImmutableSortedSetOrderedBy).* as compiler argument.

Samples

Shows the difference in example code before and after the Refaster rule is applied.

 ImmutableSortedSet.Builder<String> testImmutableSortedSetOrderedBy() {
-  return new ImmutableSortedSet.Builder<>(Comparator.comparingInt(String::length));
+  return ImmutableSortedSet.orderedBy(Comparator.comparingInt(String::length));
 }

ImmutableSortedSetNaturalOrder

SUGGESTION

Simplification

Suppression

Suppress false positives by adding the suppression annotation @SuppressWarnings("ImmutableSortedSetNaturalOrder") to the enclosing element.

Disable this rule by adding -XepOpt:Refaster:NamePattern=^(?!ImmutableSortedSetRules\$ImmutableSortedSetNaturalOrder).* as compiler argument.

Samples

Shows the difference in example code before and after the Refaster rule is applied.

 ImmutableSortedSet.Builder<String> testImmutableSortedSetNaturalOrder() {
-  return ImmutableSortedSet.orderedBy(Comparator.<String>naturalOrder());
+  return ImmutableSortedSet.naturalOrder();
 }

ImmutableSortedSetReverseOrder

SUGGESTION

Simplification

Suppression

Suppress false positives by adding the suppression annotation @SuppressWarnings("ImmutableSortedSetReverseOrder") to the enclosing element.

Disable this rule by adding -XepOpt:Refaster:NamePattern=^(?!ImmutableSortedSetRules\$ImmutableSortedSetReverseOrder).* as compiler argument.

Samples

Shows the difference in example code before and after the Refaster rule is applied.

 ImmutableSortedSet.Builder<String> testImmutableSortedSetReverseOrder() {
-  return ImmutableSortedSet.orderedBy(Comparator.<String>reverseOrder());
+  return ImmutableSortedSet.reverseOrder();
 }

ImmutableSortedSetOf

SUGGESTION

Simplification

Suppression

Suppress false positives by adding the suppression annotation @SuppressWarnings("ImmutableSortedSetOf") to the enclosing element.

Disable this rule by adding -XepOpt:Refaster:NamePattern=^(?!ImmutableSortedSetRules\$ImmutableSortedSetOf).* as compiler argument.

Samples

Shows the difference in example code before and after the Refaster rule is applied.

 ImmutableSet<ImmutableSortedSet<Integer>> testImmutableSortedSetOf() {
-  return ImmutableSet.of(
-      ImmutableSortedSet.<Integer>naturalOrder().build(),
-      Stream.<Integer>empty().collect(toImmutableSortedSet(naturalOrder())));
+  return ImmutableSet.of(ImmutableSortedSet.of(), ImmutableSortedSet.of());
 }

ImmutableSortedSetCopyOf

SUGGESTION

Simplification

Suppression

Suppress false positives by adding the suppression annotation @SuppressWarnings("ImmutableSortedSetCopyOf") to the enclosing element.

Disable this rule by adding -XepOpt:Refaster:NamePattern=^(?!ImmutableSortedSetRules\$ImmutableSortedSetCopyOf).* as compiler argument.

Samples

Shows the difference in example code before and after the Refaster rule is applied.

 ImmutableSet<ImmutableSortedSet<Integer>> testImmutableSortedSetCopyOf() {
   // XXX: The first subexpression is not rewritten (`naturalOrder()` isn't dropped). WHY!?
   return ImmutableSet.of(
-      ImmutableSortedSet.<Integer>naturalOrder().add(new Integer[] {1}).build(),
-      Arrays.stream(new Integer[] {2}).collect(toImmutableSortedSet(naturalOrder())),
-      ImmutableSortedSet.copyOf(naturalOrder(), ImmutableList.of(3).iterator()),
-      ImmutableSortedSet.<Integer>naturalOrder().addAll(ImmutableSet.of(4).iterator()).build(),
-      Streams.stream(ImmutableList.of(5).iterator())
-          .collect(toImmutableSortedSet(naturalOrder())),
+      ImmutableSortedSet.copyOf(new Integer[] {1}),
+      ImmutableSortedSet.copyOf(new Integer[] {2}),
+      ImmutableSortedSet.copyOf(ImmutableList.of(3).iterator()),
+      ImmutableSortedSet.copyOf(ImmutableSet.of(4).iterator()),
+      ImmutableSortedSet.copyOf(ImmutableList.of(5).iterator()),
       ImmutableSortedSet.copyOf(naturalOrder(), ImmutableList.of(6)),
-      ImmutableSortedSet.<Integer>naturalOrder().addAll(ImmutableSet.of(7)).build(),
-      ImmutableSortedSet.<Integer>naturalOrder().addAll(ImmutableSet.of(8)::iterator).build(),
-      Streams.stream(ImmutableList.of(9)::iterator).collect(toImmutableSortedSet(naturalOrder())),
-      ImmutableList.of(10).stream().collect(toImmutableSortedSet(naturalOrder())));
+      ImmutableSortedSet.copyOf(ImmutableSet.of(7)),
+      ImmutableSortedSet.copyOf(ImmutableSet.of(8)::iterator),
+      ImmutableSortedSet.copyOf(ImmutableList.of(9)::iterator),
+      ImmutableSortedSet.copyOf(ImmutableList.of(10)));
 }

StreamCollectToImmutableSortedSetNaturalOrder

SUGGESTION

Simplification

Suppression

Suppress false positives by adding the suppression annotation @SuppressWarnings("StreamCollectToImmutableSortedSetNaturalOrder") to the enclosing element.

Disable this rule by adding -XepOpt:Refaster:NamePattern=^(?!ImmutableSortedSetRules\$StreamCollectToImmutableSortedSetNaturalOrder).* as compiler argument.

Samples

Shows the difference in example code before and after the Refaster rule is applied.

 ImmutableSortedSet<Integer> testStreamCollectToImmutableSortedSetNaturalOrder() {
-  return ImmutableSortedSet.copyOf(Stream.of(1).iterator());
+  return Stream.of(1).collect(toImmutableSortedSet(naturalOrder()));
 }