On 2013-10-14 09:42, Staffan Larsen wrote:
Looks good!
Thanks, Staffan.
But I wonder why there is a shell script involved. Couldn't this be
just a straightforward java file with several @run tags?
Probably. I'd rather not change that now.
thanks,
StefanK
Thanks,
/Staffan
On 11 okt 201
Looks good! But I wonder why there is a shell script involved. Couldn't this be
just a straightforward java file with several @run tags?
Thanks,
/Staffan
On 11 okt 2013, at 20:02, Stefan Karlsson wrote:
> On 10/11/13 7:44 PM, Coleen Phillimore wrote:
>>
>> Stefan,
>>
>> I added a check about
On 10/11/13 8:04 PM, Coleen Phillimore wrote:
Looks good.
Thanks, Coleen.
StefanK
Thanks!
Coleen
On 10/11/2013 2:02 PM, Stefan Karlsson wrote:
On 10/11/13 7:44 PM, Coleen Phillimore wrote:
Stefan,
I added a check about the count exceeding 1 000 000 because it
causes the test to create
Looks good.
Thanks!
Coleen
On 10/11/2013 2:02 PM, Stefan Karlsson wrote:
On 10/11/13 7:44 PM, Coleen Phillimore wrote:
Stefan,
I added a check about the count exceeding 1 000 000 because it causes
the test to create an invalid class file. With the shell script
changes, it won't happen bu
On 10/11/13 7:44 PM, Coleen Phillimore wrote:
Stefan,
I added a check about the count exceeding 1 000 000 because it causes
the test to create an invalid class file. With the shell script
changes, it won't happen but I would be happy if that check were there
so no one ever has to debug thi
Stefan,
I added a check about the count exceeding 1 000 000 because it causes
the test to create an invalid class file. With the shell script
changes, it won't happen but I would be happy if that check were there
so no one ever has to debug this thinking it has a corrupted constant
pool ag
Please, review this test fix.
http://cr.openjdk.java.net/~stefank/7196801/webrev.00/
Most of the fix was implemented by Coleen, I've only done some small
tweaks and updated some comments.
The fix depends on these changes:
JDK-8025996 Track metaspace usage when metaspace is expanded
JDK-80
