On 06/02/2016 07:04 PM, Mandy Chung wrote:

Nit: on GetCallerClassTest.java:

178                     Class<?> c = (Class<?>) walker.getCallerClass();

The cast should not be needed. Any reason there?

Uh, the reason there is careless copy/paste... :$
Cast removed.

I wonder
if you could move the lambdaRunnable to a different class that the
test will fail without your fix.

Done.  Confirmed that LambdaTest passes with my fix, fails without.

Updated webrev:
http://cr.openjdk.java.net/~bchristi/8152893/webrev.01/

Thanks,
-Brent

Reply via email to