[PATCH] 8188240: Reflection Proxy should skip static methods

mandy chung mandy.chung at oracle.com
Wed Mar 14 18:00:05 UTC 2018



On 3/13/18 5:16 PM, David Lloyd wrote:
> OK, done.  It's a little bigger now so I'm attaching it.

Thanks for adding the new test.   Looks okay and some minor comment.

+        try {
    :
+        } catch (Throwable e) {
+            System.err.println("\nTEST FAILED:");
+            e.printStackTrace();
+            throw new RuntimeException("TEST FAILED: " + e.toString());
+        }


You can take out this try-catch (Basic1.java isn't be the best example to reference that is old and needs update).

I assume your colleague at Red Hat will sponsor it for you?

Mandy



More information about the core-libs-dev mailing list