Re: Code Review Request, 8144313, Test SessionTimeOutTests can be timeout

[Sean Mullan]

serverReady doesn't need to be volatile anymore.

Looks good otherwise.

--Sean

On 12/01/2015 05:48 AM, Xuelei Fan wrote:

Hi,

Please review this test update:

    http://cr.openjdk.java.net/~xuelei/8144313/webrev.00/

In test/javax/net/ssl/SSLSession/SessionTimeOutTests.java, the update of
"serverReady" variable was not synchronized when performing multiple
operations, as may result that the variable cannot be calculated
correctly, and result in timeout accordingly.

Update to use AtomicInteger for the atomic update.

Thanks,
Xuelei