> On Feb 28, 2018, at 8:29 PM, Paul Sandoz <[email protected]> wrote: > > > >> On Feb 28, 2018, at 1:22 PM, Lance Andersen <[email protected] >> <mailto:[email protected]>> wrote: >> >>> >>> On Feb 28, 2018, at 2:20 PM, Lance Andersen <[email protected] >>> <mailto:[email protected]>> wrote: >>> >>> Hi Paul, >>> >>> Thank you for the review. >>>> On Feb 28, 2018, at 1:40 PM, Paul Sandoz <[email protected] >>>> <mailto:[email protected]>> wrote: >>>> >>>> Compatible module refactoring in action! >>>> >>>> Looks good, one comment: >>>> >>>> test/jdk/javax/transaction/xa/testng/JavaSqlModuleDriver.java >>>> >>>> This is not a valid Java source file can you merge the jtreg meta data >>>> into XAExceptionTests instead? >>> >>> As we discussed offline, I will change the above file and Driver.java. I >>> naively assumed this was OK as the change to add Driver.java was made >>> after I had originally added these tests in 2015. >> >> http://cr.openjdk.java.net/~lancea/8197533/webrev.01/ >> <http://cr.openjdk.java.net/~lancea/8197533/webrev.01/> has the updated tests > > +1, i second Joe’s request to update package-info.java while we are > opportunistically cleaning this area up.
Per your and Joe’s request, please see cr.openjdk.java.net/~lancea/8197533/webrev.02 which has your suggested update to use package-info.java Thank you both Best Lance > > Paul. <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 [email protected] <mailto:[email protected]>
