Ping again - would somebody find some time to review those, please?

Thanks
M.


On 11/01/16 12:14, Miroslav Kos wrote:
Ping ... anybody?

On 22/12/15 14:44, Miroslav Kos wrote:
Hi everybody,
I'd like to ask for reviewing following patch. It fixes two issues:

JBS: https://bugs.openjdk.java.net/browse/JDK-8145104
NPE is thrown when JAXBContextFactory implementation is specified in system property

In short, it's a regression of recent changes in JAXB-AP. The old version of API is using static methods to create JAXBContext instance, where as the new configuration facility uses defined interface. The methods are called via core reflection, so when invoking non-static method of spi implemetation, instance of the spi must be created first. This fixes NPE which occured in certain scenarios.

JBS: https://bugs.openjdk.java.net/browse/JDK-8145112
newInstance(String, ClassLoader): java.lang.JAXBException should not be wrapped as expected according to spec

There are still some changes in javadoc being discussed, but anyway this double-wrapping exception into JAXBException doesn't make any sense. JAXBException thrown from JAXB spi provider is re-thrown as is.

Fix: http://cr.openjdk.java.net/~mkos/8145104/jaxws.01/
Tests: http://cr.openjdk.java.net/~mkos/8145104/jdk.02/

Added 2 new jtreg tests, the tests added also to standalone project.

Thanks
Miran



Reply via email to