From da82ae0d72bc7a157f050288caa392e8bb1a3cd7 Mon Sep 17 00:00:00 2001
From: Rick Ossendrijver <rick.ossendrijver@gmail.com>
Date: Tue, 24 Jan 2023 14:48:22 +0100
Subject: [PATCH] Suggestion

---
 .../picnic/errorprone/refasterrules/ReactorRules.java  | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/error-prone-contrib/src/main/java/tech/picnic/errorprone/refasterrules/ReactorRules.java b/error-prone-contrib/src/main/java/tech/picnic/errorprone/refasterrules/ReactorRules.java
index a19cfd2327..c99c77482a 100644
--- a/error-prone-contrib/src/main/java/tech/picnic/errorprone/refasterrules/ReactorRules.java
+++ b/error-prone-contrib/src/main/java/tech/picnic/errorprone/refasterrules/ReactorRules.java
@@ -474,11 +474,11 @@ Flux<S> after(
    * Flux}.
    */
   abstract static class MonoFlatMapToFlux<T, S> {
-    // XXX: I'd be more expressive if this `@Placeholder` were replaced with a `Function<? super T,
-    // ? extends Mono<? extends S>>` parameter, so that compatible non-lambda expression arguments
-    // to `flatMapMany` are also matched. However, the type inferred for lambda and method reference
-    // expressions passed to `flatMapMany` appears to always be `Function<T, Publisher<? extends
-    // S>>`, which doesn't match. Find a solution.
+    // XXX: It would be more expressive if this `@Placeholder` were replaced with a `Function<?
+    // super T, ? extends Mono<? extends S>>` parameter, so that compatible non-lambda expression
+    // arguments to `flatMapMany` are also matched. However, the type inferred for lambda and method
+    // reference expressions passed to `flatMapMany` appears to always be `Function<T, Publisher<?
+    // extends S>>`, which doesn't match. Find a solution.
     @Placeholder(allowsIdentity = true)
     abstract Mono<S> transformation(@MayOptionallyUse T value);