On 21/03/2014 19:50, Brian Burkhalter wrote:
On Mar 21, 2014, at 12:44 PM, Staffan Larsen<staffan.lar...@oracle.com> wrote:
Looks good to me.
Thanks.
The version with JNI_TRUE/JNI_FALSE is more correct, but I’m fine with either
one.
Updated to this form:
http://cr.openjdk.java.net/~bpb/8038163/webrev.01/
(note: this review request should have been sent to serviceability-dev).
I had no clue to which comp/sub-comp it belongs.
What test would I run to verify this change (other than that if fixes the
build)?
Looks good to me too. The tests to run are the Java Debug Interface
(JDI) tests, they are in the jdk_jdi jtreg group.
-Alan.
