RFR: 8176508 Update JAX-WS RI integration to latest version

Lance Andersen lance.andersen at oracle.com
Tue Mar 14 01:27:46 UTC 2017


Hi all,

Additional minor comments on top of what others already provided

MimetypesFileTypeMap.java

	- Are the Parens around lines 54-57 really needed?
        - defaultType and confDir, shouldn’t these be all caps like PROG?
        - For the doPrivileged method, not sure the  minimum JDK version you want to be compatible with, but you could use a lambda or specify the specific permission 
ModuleUtil.java
        -  The BufferedReader I would take advantage of try with resources so you do not have to specify finally


> On Mar 12, 2017, at 10:39 AM, Roman Grigoriadi <roman.grigoriadi at oracle.com> wrote:
> 
> Hi,
> 
> Please review standalone JAXB/JAXWS changes, synced to jdk/jaxws repo.
> 
> JBS: https://bugs.openjdk.java.net/browse/JDK-8176508
> Webrev: http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8176508/00/
> 
> Summary of changes:
> 
> jaxws/src/java.activation/share/classes/javax/activation/*
> These are from Bill, fixing JDK-8049379
> 
> jaxws/src/java.xml.bind/share/classes/javax/xml/bind/*
> JDK-8169496 - JAXB annotated classes needs to be open for JAXB impl module. Javadoc updates + propagation of openness if JAXB impl is in other module than java.xml.bind. This needs an update to JAXB JCK tests.
> 
> jaxws/src/java.xml.ws/share/classes/com/sun/xml/internal/messaging/saaj/**
> Contains a fix for reported bug after removal dependencies to JDK internal classes from SAAJ + formattig updates according to review comments on last sync.
> 
> jaxws/src/java.xml.ws/share/classes/**/*Messages.java
> jaxws/src/jdk.xml.ws/share/classes/**/*Messages.java
> These are generated files, which were modified manually in JDK in order to solve JDK-8153944. Generation of these file was updated (removed Lambda), these changes should respect JDK-8153944 fix.
> 
> Patch also contains several small bugfixes, not tracked in JBS.
> 
> Best regards,
> Roman

 <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
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>





More information about the core-libs-dev mailing list