CFR: javax.xml.parsers: Using ServiceLoader to load JAXP parser factories (7169894: JAXP Plugability Layer: using service loader)
Alan Bateman
Alan.Bateman at oracle.com
Tue Dec 11 13:01:02 UTC 2012
On 10/12/2012 17:12, Daniel Fuchs wrote:
> Hi,
>
> After further discussion with Joe & Alan, I changed the call
> to ServiceLoader to simply return the first provider it finds,
> or null.
>
> This is closer to what was present in JDK 7 - and looping is
> not necessary in JDK 8 since the default implementation is
> not returned as a service provider.
>
> So here is a new - and hopefully simpler webrev:
>
> <http://cr.openjdk.java.net/~dfuchs/JDK-7169894/javax.xml.parsers/webrev.04/>
>
>
> best regards,
>
> -- daniel
This looks fine to me, just a stray <p></p> at L64.
-Alan
More information about the core-libs-dev
mailing list