Given the changes in PublishingBodySubscriber.onError, do we still need this null check?
884 private void signalError(Throwable err) { 885 if (err == null) { 886 err = new NullPointerException("null throwable"); 887 } 888 completionCF.completeExceptionally(err); 889 } > On 13 Jun 2019, at 09:39, Chris Hegarty <chris.hega...@oracle.com> wrote: > > The convenience BodySubscriber implementations in > HttpResponse.BodySubscribers should consistently > throw NPE when passed a null value. > > https://cr.openjdk.java.net/~chegar/8225583/webrev.00/ > > -Chris.