Remove deprecated method in ClientRSocketFactory (rsocket#701)

rstoyanchev · robertroeser · commit dd1e44fe07dc · 2019-10-16T10:41:53.000-07:00
I just discovered the BiConsumer vs the SocketAcceptor variants confuse
the compiler with lambda syntax. The deprecation shouldn't be necessary
anymore.

Signed-off-by: Rossen Stoyanchev &lt;rstoyanchev@pivotal.io&gt;
diff --git a/rsocket-core/src/main/java/io/rsocket/RSocketFactory.java b/rsocket-core/src/main/java/io/rsocket/RSocketFactory.java
@@ -44,7 +44,6 @@
 import io.rsocket.util.MultiSubscriberRSocket;
 import java.time.Duration;
 import java.util.Objects;
-import java.util.function.BiFunction;
 import java.util.function.Consumer;
 import java.util.function.Function;
 import java.util.function.Supplier;
@@ -277,17 +276,7 @@ public ClientTransportAcceptor acceptor(Function<RSocket, RSocket> acceptor) {
     }
 
     public ClientTransportAcceptor acceptor(Supplier<Function<RSocket, RSocket>> acceptor) {
-      return acceptor(
-          (SocketAcceptor)
-              (setup, sendingSocket) -> Mono.just(acceptor.get().apply(sendingSocket)));
-    }
-
-    @Deprecated
-    public ClientTransportAcceptor acceptor(
-        BiFunction<ConnectionSetupPayload, RSocket, RSocket> biAcceptor) {
-      return acceptor(
-          (SocketAcceptor)
-              (setup, sendingSocket) -> Mono.just(biAcceptor.apply(setup, sendingSocket)));
+      return acceptor((setup, sendingSocket) -> Mono.just(acceptor.get().apply(sendingSocket)));
     }
 
     public ClientTransportAcceptor acceptor(SocketAcceptor acceptor) {
