Re: RFR(M): 8148195: Some InstanceKlass and MethodCounters fields can be excluded when JVMTI is not supported

2016-04-04 Thread Chris Plummer
Ping! Still looking for one more reviewer. thanks, Chris On 3/31/16 9:31 AM, Chris Plummer wrote: Hi Serguei, Thanks for the review. I'll make all the changes you suggested. cheers, Chris On 3/31/16 12:38 AM, serguei.spit...@oracle.com wrote: Hi Chris, It looks pretty good, thanks!

Re: RFR(xs): 8153396: Remove unused ResourceMarks from UL internal vm tests

2016-04-04 Thread Stefan Karlsson
Looks good. StefanK On 2016-04-04 17:09, Robbin Ehn wrote: Hi all, Please review, Webrev: http://cr.openjdk.java.net/~rehn/8153396/webrev/ Bug: https://bugs.openjdk.java.net/browse/JDK-8153396 Tested with internal vm test. Thanks! /Robbin

RFR(xs): 8153396: Remove unused ResourceMarks from UL internal vm tests

2016-04-04 Thread Robbin Ehn
Hi all, Please review, Webrev: http://cr.openjdk.java.net/~rehn/8153396/webrev/ Bug: https://bugs.openjdk.java.net/browse/JDK-8153396 Tested with internal vm test. Thanks! /Robbin