From f68492e5129c6624758194c28de301767c7516cc Mon Sep 17 00:00:00 2001 From: Stephan Schroevers Date: Sat, 15 Jan 2022 12:30:07 +0100 Subject: [PATCH] Suggestion --- .../errorprone/refastertemplates/ReactorTemplates.java | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/error-prone-contrib/src/main/java/tech/picnic/errorprone/refastertemplates/ReactorTemplates.java b/error-prone-contrib/src/main/java/tech/picnic/errorprone/refastertemplates/ReactorTemplates.java index c58b67f7a4..3f93e629fd 100644 --- a/error-prone-contrib/src/main/java/tech/picnic/errorprone/refastertemplates/ReactorTemplates.java +++ b/error-prone-contrib/src/main/java/tech/picnic/errorprone/refastertemplates/ReactorTemplates.java @@ -21,7 +21,6 @@ import reactor.core.publisher.Mono; import reactor.test.StepVerifier; import reactor.test.publisher.PublisherProbe; -import tech.picnic.errorprone.bugpatterns.FluxFlatMapUsageCheck; /** Refaster templates related to Reactor expressions and statements. */ final class ReactorTemplates { @@ -157,11 +156,8 @@ Flux after(Flux flux, Function } /** - * Prefer {@link Flux#concatMapIterable(Function)} over {@link Flux#concatMapIterable(Function)} - * to be consistent with {@link FluxFlatMapUsageCheck}. - * - *

NB: Both implementations emit values in a deterministic order and there is no difference - * with eager or lazy inner subscriptions. This means that both implementations are *equivalent*. + * Prefer {@link Flux#concatMapIterable(Function)} over {@link Flux#concatMapIterable(Function)}, + * as the former has equivalent semantics but a clearer name. */ static final class FluxConcatMapIterable { @BeforeTemplate