RFR: (JAXP) 8020430 : NullPointerException in xml sqe nightly result on 2013-07-12

Chris Hegarty chris.hegarty at oracle.com
Fri Jul 12 09:44:38 UTC 2013


The source changes look fine to me.

The tests should be located elsewhere, otherwise they will collide with an Oracle internal repo. Say jdk/test/jaxp?

As an aside, I plan to move the jaxws tests to a similar holding area, jdk/test/jaxws. They can then follow the appropriate package structure.

-Chris

On 12 Jul 2013, at 09:59, huizhe wang <huizhe.wang at oracle.com> wrote:

> Hi,
> 
> This is a quick fix on a regression caused by 8016648.  The new property manager added to handle the new jaxp 1.5 properties is only created for the input and event factories. For the output factory therefore, there should have a null-check.  I've run the tests used in the nightly build with a local build, all passed with this patch.
> 
> webrev:
> http://cr.openjdk.java.net/~joehw/jdk8/8020430/webrev/
> 
> Thanks,
> Joe



More information about the core-libs-dev mailing list