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
