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
On 12/7/12 7:37 PM, Mandy Chung wrote:
On 12/7/12 8:32 AM, Alan Bateman wrote:
On 07/12/2012 15:15, Daniel Fuchs wrote:
Hi Alan,
I have updated the webrev according to your suggestion. I think it makes
things much clearer.
The new version is there:
<http://cr.openjdk.java.net/~dfuchs/JDK-7169894/javax.xml.parsers/webrev.03/>
This looks good to me except that "implementation system default"
should be "system-default implementation".
Looks good to me too with the change to "system-default implementation".
Mandy