RFR: 8154719: JvmtiBreakpoint rename method print() to print_on()

serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Tue Apr 26 08:48:40 UTC 2016


Hi Robin,

Looks good.

Thanks,
Serguei


On 4/20/16 04:47, Robbin Ehn 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



More information about the serviceability-dev mailing list