Re: RFR JDK-8240921: Minor correction to HttpResponse.BodySubscribers example

2020-03-19 Thread Daniel Fuchs
Hi Rahul, This looks good to me. Thanks for fixing it! best regards, -- daniel On 19/03/2020 10:29, rahul.r.ya...@oracle.com wrote: Hello,  Can I please have my fix reviewed for issue JDK-8240921 - Minor correction to HttpResponse.BodySubscribers example?  The fix updates the incorrect

Re: RFR JDK-8240921: Minor correction to HttpResponse.BodySubscribers example

2020-03-19 Thread Chris Hegarty
Rahul, > On 19 Mar 2020, at 10:29, rahul.r.ya...@oracle.com wrote: > > Hello, > > Can I please have my fix reviewed for issue JDK-8240921 - Minor correction > to HttpResponse.BodySubscribers example? > > The fix updates the incorrect example of the usage of the class > java.net.http.HttpRe

Re: RFR JDK-8240921: Minor correction to HttpResponse.BodySubscribers example

2020-03-19 Thread Julia Boes
Hi Rahul, The fix looks good to me. Cheers, Julia

RFR JDK-8240921: Minor correction to HttpResponse.BodySubscribers example

2020-03-19 Thread rahul . r . yadav
Hello,  Can I please have my fix reviewed for issue JDK-8240921 - Minor correction to HttpResponse.BodySubscribers example?  The fix updates the incorrect example of the usage of the class java.net.http.HttpResponse.BodySubscribers.The patch also updates the test case for this scenario.  I

RFR JDK-8240921: Minor correction to HttpResponse.BodySubscribers example

2020-03-19 Thread rahul . r . yadav
Hello,  Can I please have my fix reviewed for issue JDK-8240921 - Minor correction to HttpResponse.BodySubscribers example?  The fix updates the incorrect example of the usage of the class java.net.http.HttpResponse.BodySubscribers.The patch also updates the test case for this scenario.  I

RFR JDK-8240921: Minor correction to HttpResponse.BodySubscribers example

2020-03-19 Thread rahul . r . yadav
Hello,  Can I please have my fix reviewed for issue JDK-8240921 - Minor correction to HttpResponse.BodySubscribers example?  The fix updates the incorrect example of the usage of the class java.net.http.HttpResponse.BodySubscribers.The patch also updates the test case for this scenario.  I