[modules-dev] Review request for 6578988
Dave Bristor
David.Bristor at Sun.COM
Tue Oct 9 13:28:47 PDT 2007
Hi folks,
This fix for
(repo) Implement Repository.shutdownOnExit
builds on top of my previous request (attached); it has build atop that
because the changes here for shutdownOnExit() rely on the changes to
shutdown() in the prior request.
This is a *lot* shorter than my previous request. It includes some javadoc
fixes in JamUtils.
You'll see in the change that when shutdownOnExit is invoked with false, that
any previously-created shutdownThread is made available to GC. I didn't see
any reason to retain the reference if the likelihood is that it won't be used.
It might be better to keep it once created. Space v. time!
bugster: http://monaco.sfbay/detail.jsp?cr=6578988
webrev: http://analemma.sfbay.sun.com/java/jdk/ws/libs/rev/6578988
I submitted a JPRT run that has changes for this fix plus the others.
Thanks!
Dave
-------------- next part --------------
An embedded message was scrubbed...
From: Dave Bristor <David.Bristor at Sun.COM>
Subject: Review request for 6559067, 6559086, and 6605100
Date: Fri, 05 Oct 2007 15:55:54 -0700
Size: 3205
Url: http://mail.openjdk.java.net/pipermail/modules-dev/attachments/20071009/f2e03877/attachment.mht
More information about the modules-dev
mailing list