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