Hi Pavel,

Thanks for review, i updated the webrev(http://cr.openjdk.java.net/~vtewari/8144692/webrev0.0/index.html <http://cr.openjdk.java.net/%7Evtewari/8144692/webrev0.0/index.html>) in place.

Thanks,
Vyom


On Tuesday 12 July 2016 06:10 PM, Pavel Rappo wrote:
Hi Vyom,

A minor comment. Do you think it would make sense to use space after

     new InetSocketAddress(8000),
as in every other line in this javadoc where multiple arguments passed, the
space is used. Just to be consistent in formatting.

Thanks.

On 12 Jul 2016, at 12:36, Vyom Tewari <vyom.tew...@oracle.com> wrote:

Hi All,

Please review below small doc fix.

Bug            : JDK-8144692 HttpServer API: use of non-existant method in 
example in package Javadoc
Webrev      : http://cr.openjdk.java.net/~vtewari/8144692/webrev0.0/index.html 
<http://cr.openjdk.java.net/%7Evtewari/8144692/webrev0.0/index.html>

Thanks,
Vyom

Reply via email to