Re: RFR(XS): 8056994: sun/tools/jstatd/TestJstatdPortAndServer.java and sun/tools/jstatd/TestJstatdServer.java miss correct check of RMI server availability

[Yekaterina Kantserova]

On 09/01/2014 03:47 PM, Jaroslav Bachorik wrote:

Hi Katja,

On 09/01/2014 02:04 PM, Yekaterina Kantserova wrote:

Hi,

Could I please have a review of this small fix.

webrev: http://cr.openjdk.java.net/~ykantser/8056994/webrev.00/
bug: https://bugs.openjdk.java.net/browse/JDK-8056994

I would recommend moving the code from L154-157 before the start of the while loop and simplify it with

`String remoteHost = serverName != null ? serverName : "JStatRemoteHost"`

Ouch, has missed while loop. Thanks for pointing it out! Will fix.

// Katja

Cheers,

-JB-

Thanks,
Katja