RFR [7u8] (JAXP): 7191547 : XMLEventFactory.newFactory(String factoryId, ClassLoader loader) does not work as expected
Joe Wang
huizhe.wang at oracle.com
Fri Aug 17 04:07:55 UTC 2012
In the patch for 6756677, we fixed errors in StAX's input and output
factories where factoryId such as 'javax.xml.stream.XMLInputFactory" was
taken as factory class e.g.
com.sun.xml.internal.stream.XMLInputFactoryImpl. Unfortunately, we
missed XMLEventFactory in that patch.
So this is a patch for XMLEventFactory similar to that of
XMLInputFactory and XMLOutputFactory. The change itself is to search an
implementation class using the provided factory id instead of trying to
create a instance directly.
The webrev is here: http://cr.openjdk.java.net/~joehw/7u8/7191547/webrev/
All tests pass except two XMLEventFactory tests that assumed the
factoryId to mean factory class. They will be updated once this patch is
approved.
Thanks,
Joe
More information about the core-libs-dev
mailing list