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? Thanks, -Aleksey