Hi David,
Thanks for your comments, I have updated the fix based on your
suggestion, Can you please help to review again.
http://dl.dropbox.com/u/90659131/fixes/6959636/webrev/index.html
Thanks,
Eric
On 2012/9/3 11:45, David Holmes wrote:
Hi Eric,
On 3/09/2012 1:22 PM, Eric Wang wrote:
Hi All,
I am an engineer from SQE team who is helping to cleanup ProblemList.
Here is a fix below for bug 6959636
<http://monaco.us.oracle.com/detail.jsf?cr=6959636>, Can you please help
to review?
http://dl.dropbox.com/u/90659131/fixes/6959636/webrev/index.html
The test failed when it is executed by the command "jtreg -agentvm
-retain:fail,error", root cause is "-retain:fail,error" option tries to
remove temp files created by test but they are still opened by the
agentvm/samevm.
So the fix for that is to run in othervm?
And the code changes are actually nothing to do with the original
problem? You are just simplifying the error logic by having the
exceptions propagate?
Here:
130 System.err.println("Failed to load the MBean #" + j +
131 ". The shown Throwable was caught.");
and here:
145 } catch (ReflectionException e) {
146 System.err.println("A ReflectionException, wrapping " +
147 "the shown exception, occured when" +
The printed messages no longer makes sense because there is no "shown
exception" anymore. I'd suggest just getting rid of the println's and
let the exceptions propagate directly - no need for any try/catch now
that main throws Throwable.
David
Thanks,
Eric
