Hey Stuart,
the changes look good to me, although I didn't apply the patch and
run the tests. The only comment I have is to update all copyrights to 2013.
Darryl
On 01/03/2013 06:39 PM, Stuart Marks wrote:
Hi all,
Please review these additional RMI test cleanups:
http://cr.openjdk.java.net/~smarks/reviews/7187882/webrev.0/
in service of fixing the following bug:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7187882
The basic problem here is that tests were waiting for the JVM
subprocess to exit, but not waiting for all output from the subprocess
to be collected. The fix includes a bit of new infrastructure in RMI's
test library and adjustment of several other tests to use it.
Thanks,
s'marks
