Hi Dmitry,

Looks good in general.

The following files in the jdk.hotspot.agent/test folder may need to be updated too:
  src/jdk.hotspot.agent/test/jdi/runjdb.sh
  src/jdk.hotspot.agent/test/jdi/runsa.sh

Thanks,
Serguei


On 3/29/16 02:41, Alan Bateman wrote:


On 29/03/2016 09:54, Dmitry Samersoff wrote:
Alan,

done (in-place)

http://cr.openjdk.java.net/~dsamersoff/JDK-8150973/webrev.02/

-Dmitry


Looks okay to me. At some point (and nothing to do with this change) is that we should add type parameters to the other methods like getPath, classPath, etc.

-Alan

Reply via email to