Jigsaw Enhancement RFR round #3: 8159145 Add JVMTI function GetNamedModule

Alan Bateman Alan.Bateman at oracle.com
Tue Jul 5 13:11:01 UTC 2016


Just catching up on this thread, I assume this is the current patch:

http://cr.openjdk.java.net/~sspitsyn/webrevs/2016/hotspot/8159145-jigsaw-jvmti-pkg.6/

For the description then it might be clearer to say "for a named module" 
rather than "for a module" so that the first paragraph is clear than 
this is function to obtain a reference to a named module.

For the class_loader description then it might be clearer to say is not 
NULL and a not a subclass of ...

At some point then I assume the version="9.0.0" changes should be 
collapsed into one change element.

The rest looks okay to me.

-Alan




More information about the serviceability-dev mailing list