Hi all,

Here's revised webrev, as Alan suggested, including the implementation.

http://cr.openjdk.java.net/~joehw/jdk8/jaxp16MR/8028822/webrev/

Thanks,
Joe

On 11/26/2013 8:09 AM, Alan Bateman wrote:
On 25/11/2013 21:51, huizhe wang wrote:
Hi,

This is a patch to fix an error in StAX factories' newFactory(String factoryId, ClassLoader classLoader). In the 3 step of the documentation, it should have stated that the specified ClassLoader is used.

http://cr.openjdk.java.net/~joehw/jdk8/jaxp16MR/webrev/
The change looks good to me in that it makes these strange factory methods somewhat usable.

-Alan

Reply via email to