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

2016-04-04 Thread Robbin Ehn
Thanks Stefan! /Robbin On 04/04/2016 05:23 PM, Stefan Karlsson wrote: 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 te

[PATCH resend] 8145278: Fix memory leak in splitPathList

2016-04-04 Thread Alex Henrie
# HG changeset patch # User Alex Henrie # Date 1447827030 25200 # Tue Nov 17 23:10:30 2015 -0700 # Node ID 8021a03115817aa9c641333848e2f483516f06a7 # Parent 0007e5d265a849b73114c98b3d4f55dde83a57ea 8145278: Fix memory leak in splitPathList diff --git a/src/java.instrument/share/native/libin

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! Jus

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