RFR: 8044656: Update JAX-WS RI integration to latest version (original) (raw)

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/jaxwsclasses/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/jaxwsclasses/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