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.

Paul.

Reply via email to