8067947 - Regression test for JDK-6522873

[Christian Tornqvist]

Hi Jesper,

This looks good, thanks for adding this :)

Thanks,
Christian

-----Original Message-----
From: hotspot-runtime-dev [mailto:hotspot-runtime-dev-bounces at openjdk.java.net] On Behalf Of Jesper Wilhelmsson
Sent: Monday, December 22, 2014 10:00 AM
To: Thomas Schatzl
Cc: 'hotspot-runtime-dev'
Subject: Re: 8067947 - Regression test for JDK-6522873

Hi Thomas,

Thanks for looking at this!

I fixed the copyright year and added a @bug.
In case you want to see a new webrev it is available here:
http://cr.openjdk.java.net/~jwilhelm/8067947/webrev.02/

/Jesper

Thomas Schatzl skrev 22/12/14 10:02:
> Hi Jesper,
>
> On Sat, 2014-12-20 at 02:30 +0100, Jesper Wilhelmsson wrote:
>> Hi Christian,
>>
>> Thanks for looking at this!
>>
>> You are right. My initial thought was to reuse the ErrorOnStart class 
>> in other tests but it doesn't really add any value since the provided 
>> functionality is fairly simple.
>>
>> I removed the class and changed the test according to your suggestion.
>> A new webrev is available here:
>>
>> http://cr.openjdk.java.net/~jwilhelm/8067947/webrev.01/
>
> - copy&pasted header from an old test (copyright date) ;)
>
> - no @bug tag in the test
>
> Thanks,
>    Thomas
>
>