Hi Darryl,

Thanks for looking. Was it last year already that I developed this patch? Sheesh! :-)

s'marks

On 1/4/13 9:24 AM, Darryl Mocek wrote:
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

Reply via email to