On 10/4/2013 9:36 AM, Paul Sandoz wrote:
On Oct 4, 2013, at 6:11 PM, Daniel Fuchs <daniel.fu...@oracle.com> wrote:

On 10/4/13 3:33 PM, Alan Bateman wrote:
On 04/10/2013 03:02, Daniel Fuchs wrote:
:

http://cr.openjdk.java.net/~dfuchs/webrev_8025745/webrev.00/
This mostly looks good, the only thing is that it reads "the
service-provider will use ..." but I think you mean "the
service-provider loading facility will use ..."

-Alan.
Right! Thanks for spotting that Alan.

http://cr.openjdk.java.net/~dfuchs/webrev_8025745/webrev.01

Looks good.

It would be nice to factor out the repeating text to a link in package javadoc, 
but there does not appear to be any suitable shared location; stuff on java.xml 
might work but probably will look out of place.

True. It might have been a good fit if we had a package such as javax.xml.jaxp for the core JAXP APIs, similar to javax.xml.ws that covers the core JAX-WS APIs.

Joe


Paul.

Reply via email to