RFR: 8076549: Update JAX-WS RI integration to latest version (2.2.11-b150402.1412)

huizhe wang huizhe.wang at oracle.com
Fri Apr 10 06:50:14 UTC 2015


Hi Aleksej,

I assume the JAX-WS/JAXB standalone releases are well tested for this 
integration, esp. the service finder code.

I see a great number of "<>" to "<>" changes that was suggested by 
Alan in the last integration. Besides that, you may consider removing 
</p>, using {@literal @} to replace &#64, and the diamond operator in 
new expression in MIMEMessage.  Also, new 
StringBuilder().append("META-INF/services/").append(jaxbContextFQCN).toString() 
isn't any better than "META-INF/services/"+jaxbContextFQCN since they'll 
be compiled into the same.

None of the above should prevent you from pushing your changes.

Cheers,
Joe

On 4/8/2015 1:57 PM, Aleksej Efimov wrote:
> Hi,
> Can I kindly ask for a review of this JAX-WS update?
>
> With Best Regards,
> Aleksej
>
> On 04/03/2015 06:20 PM, Aleksej Efimov wrote:
>> Hello,
>>
>> Can I have a review for a JDK9 bulk update of JAX-B/WS from upstream 
>> projects.
>> Webrev: http://cr.openjdk.java.net/~aefimov/8076549/9/00/
>> More details in issue description: 
>> https://bugs.openjdk.java.net/browse/JDK-8076549
>>
>> Thank you,
>> Aleksej
>




More information about the core-libs-dev mailing list