Review Request - JDK-6720349: (ch) Channels tests depending on hosts inside Sun
Alan.Bateman at oracle.com
Tue Nov 6 08:49:56 PST 2012
On 06/11/2012 16:10, Daniel Fuchs wrote:
> Here is a new webrev incorporating Alan's feedback.
> I fixed the indenting issues by looking at the original code
> in the modified file (or in other files in the same directory),
> and trying to guess how the author might have indented the lines
> in question. That's just guesses on my part but hopefully it's
> better now than before.
> best regards,
> -- daniel
I think this looks very good. The indenting was a bit of a side issue
anything, I think it's fine now.
On "TestClass", I just realized I suggested this to you in the first
mail. My brain was thinking "TestServers", my fingers wrote "TestClass"
for some reason and I see you've pulled it out of TestUtil with that name.
- in LocalAddress the comment reads "... hopefully it doesn't matter for
the test purposes". I think this can be removed as an echo server is
- typo in the comment in ConnectState.java (line 114). It reads " We may
get a an AlreadyConnected exception", I assume this should be "We may
Otherwise, thumbs up from me.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the nio-dev