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 @, 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