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

Roman Grigoriadi roman.grigoriadi at oracle.com
Wed Feb 15 12:40:18 UTC 2017


Hi Lance,

On 02/14/2017 09:28 PM, Lance Andersen wrote:
> Hi Roman,
>
> I made a pass through the changes.  And here are some general comments 
> which are mostly minor in nature….
>
> HTH
>
> Best
> Lance
> ----------------------
> JAXBRIContext.java:
> - The comment “If true, despite of the specification….”;  This does 
> not read well can this be clarified?
We have changed it to:
* If true and element namespace is not specified, namespace of parent 
element will be used.
* The default value is false.
Hope this is acceptable.
>
> Generated.java, PostConstruct.java, PreDestroy.java,Resource.java
>          -  Be nice to use {@code} vs <code></code> for readability
>          -  Do you really need <p></p>?  I am assuming there are no 
> doclint errors on Generated?
>          - Was there a CCC for the changes as there were some minor 
> semantic updates
These are from JSR 250 we don't own sources.
>
>
> MimeMultipart.java.:
>        - createMimeBodyPart @param len does not align properly
>
> SharedInputStream.java
>        - writeTo @param out does not align
>
> NameImpl.java
>        -  Some comments have a blank line before the params, others do 
> not.  It would be nice to be consistent
fixed
>
> ParserPool.java
>        - Can you explain the status regarding returnParser() and 
> https://jaxp.dev.java.net/issues/show_bug.cgi?id=59
removed //TODO and commented method, Aleksej will provide patch for 
resetting symbol table on parser.reset()
> SAAJUtil.java
>        - getSystemClassLoader, do you really need to check for a 
> SecurityManager?  Why not just AccessController.doPrivilege(could also 
> use a lambda also)
Removed SecurityManager check, lambda cannot be used without utilizing 
multirelease-jar for saaj, our min supported version is 1.7
>
> MethodUtil.java
>        - Looks like the alignment is off in the changes in invoke
There are 5 of them in the sync. I reformatted new ones. Can't see any 
wrong indentation in updated ones. Also fixed package in some comments/logs.
>
> ContextClassloaderLocal.java
>         -  getContextClassLoader could be changed to use a lambda with 
> AccessController.doPrivilege if you desire
>
> FactoryFinder.java
>        - getSystemProperty, it looks like you were using a lambda and 
> removed it for the AccessController.doPrivilege?
both no lambda for min target 1.7
>
> SecureLoader.java
>       - same comment as SAAJUtil.java
I don't know the reason why SecurityManager presence check is present 
there. Maybe performance? But there are 14 copies of SecureLoader class 
in JAXB, if we do remove unnecessary check we should replace all of 
them. Lets postpone it.
> NGCCRuntimeEx.java
>       - copyLocater, createValidationContext @return comment needs 
> checked
fixed
>
> package-info.java
>       - Seem the new files include a 2003/2004/1997, 2017, copyright, 
> should just be 2017 i believe
fixed

we are working on a new webrev, hopefully will post it later today.

Roman


>
>> On Feb 14, 2017, at 7:00 AM, Roman Grigoriadi 
>> <roman.grigoriadi at oracle.com <mailto: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-8174735
>> Webrev: 
>> http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8174735/00/ 
>> <http://cr.openjdk.java.net/%7Eaefimov/jaxws-integrations/8174735/00/>
>>
>> You can find change list in the description of JBS and its linked 
>> issues. Most diffs are for SAAJ-RI related to JDK-8166745 - remove 
>> dependencies on jdks Xerces internal classes.
>>
>> Thanks,
>> 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