RFR(s): 8035000: TEST_BUG: remove ActivationLibrary.DestroyThread and have callers call rmid.destroy() instead

Stuart Marks stuart.marks at oracle.com
Tue Nov 25 02:26:25 UTC 2014


Hi all,

Here's another test cleanup fix. Basically this normalizes the shutdown/destroy 
policy for rmid processes that are started by RMI's test library in support of a 
few dozen of the RMI activation tests. The previous implementation was, well, 
confused, and it had a redundant timing loop that can be subsumed by other code 
in RMI's test library (JavaVM.waitFor).

I've listed this as a "small" changeset even though lots of files have changed. 
In all but three or so of the files, the only change is from calling 
"ActivationLibrary.rmidCleanup(rmid)" to calling "rmid.cleanup()" which is a 
refactoring enabled by the cleanup. The real action is in the files 
test/java/rmi/testlibrary/ActivationLibrary.java and RMID.java.

Webrev:

     http://cr.openjdk.java.net/~smarks/reviews/8035000/webrev.0/

Bug:

     https://bugs.openjdk.java.net/browse/JDK-8035000

Thanks,

s'marks



More information about the core-libs-dev mailing list