Hi Serguei,

On 21/05/2019 4:07 am, serguei.spit...@oracle.com wrote:
Please, review a fix for enhancement:
https://bugs.openjdk.java.net/browse/JDK-8046018

Related CSR:
https://bugs.openjdk.java.net/browse/JDK-8223915

I have some comments on the CSR and about this change overall as to me it is not a simple clarification at all, but potentially a significant change in the meaning of the capability.

Webrev:
http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8046018-jvmti-cap-spec.1/

You introduced a typo: modifialble

Assuming this proceeds a similar change is needed earlier:

7444         <capability id="can_redefine_any_class">
7445 If possessed then all classes (except primitive, array, and some implementation defined
7446           classes) are modifiable (redefine or retransform).

Thanks,
David
-----


Summary:

  The fix is to make the JVMTI can_redefine_any_class capability spec more inconsistent.
   It is just about a couple of lines.

Thanks,
Serguei

Reply via email to