CFR: javax.xml.parsers: Using ServiceLoader to load JAXP parser factories (7169894: JAXP Plugability Layer: using service loader) (original) (raw)

Daniel Fuchs daniel.fuchs at oracle.com
Tue Dec 11 14:10:24 UTC 2012


On 12/11/12 2:01 PM, Alan Bateman wrote:

On 10/12/2012 17:12, Daniel Fuchs wrote: [...]

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

at L64.

Right - I originally did it on purpose because the doc was looking weird without it. But now I regenerated the doc and I don't think it looks better with it - so I will remove the stray

. I'll just refresh webrev.04 after that (I assume there's no need for a webrev.05 - but I'd like to keep a public copy of the 'final' patch).

-- daniel

-Alan



More information about the core-libs-dev mailing list