+1

On 03/01/2019 07:18, Alexey Ivanov wrote:
Hi Krishna,

On 03/01/2019 05:40, Krishna Addepalli wrote:
Hi Alexey,

Thank you for the suggestion. Here is the new webrev: 
http://cr.openjdk.java.net/~kaddepalli/8215910/webrev01/

This version looks good to me.

I'm not a reviewer though, so you'll need someone else's okay too.


Regards,
Alexey


Thanks,
Krishna

On 31-Dec-2018, at 6:39 PM, Alexey Ivanov <alexey.iva...@oracle.com 
<mailto:alexey.iva...@oracle.com>> wrote:

Hi Krishna,

But the bug hasn't been fixed yet:

- 230      * @param listener          The {@code Runnable}Runnable whose
+ 230      * @param listener          The {@code Runnable} Runnable whose

The second “Runnable” is not needed at all.

Regards,
Alexey

On 31/12/2018 07:50, Krishna Addepalli wrote:
Hi All,

Please review a trivial doc fix for JDK-8215910.
Webrev: http://cr.openjdk.java.net/~kaddepalli/8215910/webrev00/

Thanks,
Krishna





--
Best regards, Sergey.

Reply via email to