RFR [9]: 8145104: NPE is thrown when JAXBContextFactory implementation is specified in system property, 8145112: newInstance(String, ClassLoader): java.lang.JAXBException should not be wrapped as expected according to spec

[Miroslav Kos]

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
>