RFR: 8004183: test/sun/management/jmxremote/LocalRMIServerSocketFactoryTest.java doesn't clean-up

[Daniel Fuchs]

Hi Peter,

Looks good to me - but you might want to use 'boolean' for
isRunning rather than 'Boolean'.
Joining on the daemon thread is probably not necessary,
but there's no harm in it (the important part being
isRunning=false + s.close()).

best regards,

-- daniel (not a reviewer)

On 10/11/13 11:53 AM, shanliang wrote:
> Looks good to me.
>
> Shanliang
>
>
> Peter Allwin wrote:
>> Hello!
>>
>> Looking for reviews of this fix where a jmxremote test left a worker
>> thread running after completion. Fix is to flag the thread to finish
>> and join before test method exit.
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8004183
>> cr: http://cr.openjdk.java.net/~allwin/8004183/webrev.00
>>
>> Thanks!
>> /peter
>