RFR: javax.xml.stream: Using ServiceLoader to load JAXP stream factories (7169894: JAXP Plugability Layer: using service loader)

Daniel Fuchs daniel.fuchs at oracle.com
Tue Dec 18 16:39:41 UTC 2012


Hi,

Thanks for the review.
I updated the webrev to keep track of your suggested change.
<http://cr.openjdk.java.net/~dfuchs/JDK-7169894/javax.xml.stream/webrev.01/>

-- daniel

On 12/18/12 4:00 PM, Alan Bateman wrote:
>
> I looked through this installment and aside from an aside from an
> alignment issue at lines 101-102 in XMLEventFactory.java then it looks
> good to me.
>
> Also thank you again for being so careful as you work through each of
> these areas.
>
> -Alan




More information about the core-libs-dev mailing list