On 22/12/2017 10:16, Chris Hegarty wrote:

On 21/12/17 21:40, mandy chung wrote:
 ... >
ReflectionFactory or other class in sun.reflect package would do it.
and create an issue to follow up sun.reflect.Reflection as flagged as a removed API.
That’s what the test now does with my changes. Why separate it
out into a separate issue?   If we need a test for an internal API, I
can add a scenario that uses sun.reflect.ReflectionFactory.

Yes please. That's what this case intends to verify.

Webrev with the test updated as suggested:
  http://cr.openjdk.java.net/~chegar/8179424/webrev.03/
Updated webrev looks good to me too.

-Alan

Reply via email to