Hi Chris,

Well spotted.

I've made those changes and updated the webrev.

http://cr.openjdk.java.net/~pconcannon/8235783/webrevs/webrev.02/

Kind regards,

Patrick


On 10/01/2020 16:17, Chris Hegarty wrote:
Patrick.

On 10 Jan 2020, at 15:26, Patrick Concannon <patrick.concan...@oracle.com> 
wrote:

Hi Alan,

Thanks for the feedback.

I've incorporated your comments into the webrev below.

http://cr.openjdk.java.net/~pconcannon/8235783/webrevs/webrev.01/
Mostly looks good. A few small comments:

1) "If this **channel’s** socket is not bound then ...” - please remove 
“channel’s"

2) Maybe start the text of the new throws clauses with lowercase - which seems 
to be the existing style

-Chris.

Reply via email to