Staffan,

changes look good.

/R

On Mar 16, 2012, at 9:14 AM, Staffan Larsen wrote:

> Thanks Alan.
> 
> Anyone else?
> 
> /Staffan
> 
> On 15 mar 2012, at 11:41, Alan Bateman wrote:
> 
>> On 15/03/2012 09:30, Staffan Larsen wrote:
>>> Please review the following change to allow a test to be compiled that was 
>>> previously not being run.
>>> 
>>> Webrev: http://cr.openjdk.java.net/~sla/7152798/webrev.01/ 
>>> <http://cr.openjdk.java.net/%7Esla/7152798/webrev.01/>
>>> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7152798
>>> 
>>> After the change, the test compiles and passes (verified in JPRT).
>>> 
>>> Thanks,
>>> /Staffan
>>> 
>> Thanks for fixing up this test and making sure it passes everywhere. The 
>> changes look okay to me. I see some odd formatting in KlassLoader that could 
>> be sorted out too but it's not important.
>> 
>> -Alan
> 

Reply via email to