Thanks Staffan!

/Robbin

On 04/26/2016 10:18 AM, Staffan Larsen wrote:
Looks good!

Thanks,
/Staffan

On 20 apr. 2016, at 13:47, Robbin Ehn <robbin....@oracle.com> wrote:

Hi all, please review!

print() method was changed in 8154041 to have outputStream argument, this 
renames print to print_on.
Since JvmtiBreakpoint extends GrowableElement print_on already exists as a 
virtual const method, hence the const changes.

Bug: https://bugs.openjdk.java.net/browse/JDK-8154719
Webrev: http://cr.openjdk.java.net/~rehn/8154719/webrev/

Tested with:

#define TraceJVMTICalls false
in hotspot/src/share/vm/prims/jvmtiEnv.cpp
set to true with test-set nsk.jvmti.SetBreakpoint

Thanks!

/Robbin

Reply via email to