RFR: 8174735 Update JAX-WS RI integration to latest version
Roman Grigoriadi
roman.grigoriadi at oracle.com
Wed Feb 15 13:36:57 UTC 2017
Adding Lukas
On 02/15/2017 01:40 PM, Roman Grigoriadi wrote:
>
> 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