On 7/5/16 06:11, Alan Bateman wrote:
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.
Fixed.
For the class_loader description then it might be clearer to say is
not NULL and a not a subclass of ...
Fixed.
At some point then I assume the version="9.0.0" changes should be
collapsed into one change element.
Fixed.
The rest looks okay to me.
Thanks, Alan!
Serguei
-Alan
