[PATCH] Review request: 8004177: test/java/lang/Thread/GenerifyStackTraces.java doesn't clean-up
Chris Hegarty
chris.hegarty at oracle.com
Thu May 16 09:16:15 UTC 2013
On 05/16/2013 10:10 AM, Alan Bateman wrote:
> On 15/05/2013 12:07, Eric Wang wrote:
>>>
>> Hi Alan,
>>
>>
>> Thanks for the suggestion, I have updated the fix again, Can you
>> please help to take a look?
>> http://cr.openjdk.java.net/~ewang/8004177/webrev.01/
>> <http://cr.openjdk.java.net/%7Eewang/8004177/webrev.01/>
> Updated webrev looks good to me. Minor nit is that you are missing a
> spec in "}finally " at line 55.
Yes, I noticed this too. I'll fix it before pushing.
-Chris.
> I think Chris offered to sponsored this one and it would be good to get
> it in as this thread is taking cycles from other tests that run
> subsequently in the same VM.
>
> -Alan.
More information about the core-libs-dev
mailing list