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.

Reply via email to