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