David,

> On 21 Dec 2017, at 00:42, David Holmes <david.hol...@oracle.com> wrote:
> 
> Hi Chris,
> 
> Adding in hotspot-runtime-dev now that you have included the VM side of the 
> cleanup. What repo are you planning on pushing this to?

Since there are now changes in the hotspot area, then it
probably makes sense to push this through jdk/hs. 

> ...
> 
> Can you fix an existing typo please:
> !  * error if it is not marked propertly.
> propertly -> properly

Fixed.

> Also you seem to have missed this test reference:
> ./langtools/tools/jdeps/jdkinternals/src/p/Main.java:        Class<?> caller 
> = Reflection.getCallerClass(2);

Oops. Adding langtools testing too.

Updated webrev:
  http://cr.openjdk.java.net/~chegar/8179424/webrev.02/

What’s the correct set of tests and route into jdk/hs these
days? I’ve not pushed there in a while.

-Chris.

Reply via email to