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
