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.