Thanks Dan!

David

On 18/01/2017 12:27 PM, Daniel D. Daugherty wrote:
On 1/17/17 6:03 PM, David Holmes wrote:
Bug: https://bugs.openjdk.java.net/browse/JDK-8172261

webrev: http://cr.openjdk.java.net/~dholmes/8172261/webrev/

src/share/vm/prims/jvmti.xml
    No comments.

Thumbs up.

Dan




This is a simple relaxation of the spec in regards to the new "early
VM start" event, to explicitly allow for the possibility that
arbitrary class loading during this "early VM start" time may fail in
extreme ways.

Changes are easier to see in text than webrev:

Extend:

The VM is capable of executing bytecode but it may not have
initialized to the point where it can load classes in modules other
than java.base.

to read:

The VM is capable of executing bytecode but it may not have
initialized to the point where it can load classes in modules other
than java.base, or even arbitrary classes in java.base.

---

Replace:

Care should also be taken with JNI FindClass as it may not be possible
to load classes that are not in the java.base module.

with:

Extreme care should also be taken with JNI FindClass as it may not be
possible to load classes, and attempts to do so may result in
unpredictable behavior, maybe even stability issues on some VM
implementations.

---

Thanks,
David

Reply via email to