On 3/14/18 11:37 AM, Aleksey Shipilev wrote:
On 03/14/2018 07:09 PM, David Lloyd wrote:
On Wed, Mar 14, 2018 at 1:00 PM, mandy chung <mandy.ch...@oracle.com> wrote:
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).
Fixed & attached.
I assume your colleague at Red Hat will sponsor it for you?
I will find out.
I can do it, but I need to get updated on two things:
a) Are we pushing JDK changes directly to jdk/jdk now?
b) Do we need to run it through JDK Submit first, or that is optional for
this change?
While this patch is trivial, I recommend to run through JDK submit since
proxies are used by other areas. I did find surprises in the past via
testing on trivial proxy class changes.
Mandy
