On 22/03/2018 10:03, Amit Sapre wrote:

Thanks Alan and Mandy for inputs.

This webrev : http://cr.openjdk.java.net/~asapre/webrev/2018/JDK-8071367/webrev.01/ <http://cr.openjdk.java.net/%7Easapre/webrev/2018/JDK-8071367/webrev.01/> addresses your comments.

I reverted changes for  legacy.properties  and also rebased the patch for jdk/jdk repo.


This looks good, just a few typos in Agent's class description

"This class also provide entrypoints for jcmd tool ..." => "This class also provides entry points for the jcmd tool ...".

-Alan.

Reply via email to