RFR: 8044656: Update JAX-WS RI integration to latest version
Miroslav Kos
miroslav.kos at oracle.com
Wed Jul 9 11:13:12 UTC 2014
Hi Joe,
thanks for comments - see inline:
On 08/07/14 01:02, huizhe wang wrote:
> Hi Miran,
>
> In class src/share/jaxws_classes/com/sun/tools/internal/xjc/Options.java:
> + // hack to force initialization so catalog manager system
> properties take effect
> + staticManager.getVerbosity();
According to Iaroslav (JAXB lead), the initilization is necessary. It
looks like system property is used only if initialization performed via
getter.
>
> Is that necessary? I thought the next call to getCatalog would
> initialize the properties from the CatalogManager property file.
>
> Using ServiceLoader in class
> src/share/jaxws_classes/javax/xml/ws/spi/Provider.java:
> Would you need to give the process privilege? Or maybe the method is
> already called in a privileged block?
I think it doesn't - the previous version was exactly same, the only
difference was that it was called via reflection.
M.
>
> Thanks,
> Joe
>
> On 6/30/2014 9:22 AM, Miroslav Kos wrote:
>> Hi,
>> there is a bulk update of JAX-B/WS from upstream projects -
>> webrev: http://cr.openjdk.java.net/~mkos/8044656/jaxws.00/
>> more details in issue desc:
>> https://bugs.openjdk.java.net/browse/JDK-8044656
>>
>> Could I ask for a review/approval?
>>
>> Thanks
>> Miran
>
More information about the core-libs-dev
mailing list