[8u20] RFA: 8032392 : Spec: javax.xml.stream.XMLEventFactory/XMLOutputFactory/XMLInputFactory.newFactory(String, ClassLoader) referring to ServiceLoader.load(Class, ClassLoader)

[huizhe wang]

Thanks all!

Joe

On 1/28/2014 6:51 AM, Seán Coffey wrote:
> Thanks for comments Joe/Paul.
>
> Joe Wang - consider this approved for jdk8u-dev then!
>
> regards,
> Sean.
>
> On 27/01/2014 23:50, Joe Darcy wrote:
>> Hello,
>>
>> I agree with Paul's assessment of the fix; since there is only one 
>> ServiceLoader method has a ClassLoader argument, it is is just a 
>> clarification to state that is the method which is called.
>>
>> HTH,
>>
>> -Joe
>>
>> On 01/27/2014 08:19 AM, Jeff Dinkins wrote:
>>>
>>> Joe - do you have any additional thoughts?
>>>
>>> On Jan 27, 2014, at 3:45 AM, Paul Sandoz <paul.sandoz at oracle.com> 
>>> wrote:
>>>> Hi Sean,
>>>>
>>>> IMHO in general clarifications to docs (no changes to specs) are 
>>>> fine. We should not treat docs any different to code in this 
>>>> respect. The right fix is to support re-publishing.
>>>>
>>>> I looked at 8032392 and one could reasonable argue this is not a 
>>>> spec change, since the behaviour cannot be otherwise (there is only 
>>>> one ServiceLoader.load method that accepts a ClassLoader).
>>>>
>>>> Paul.
>>>>
>>>> On Jan 25, 2014, at 12:21 PM, Seán Coffey <sean.coffey at oracle.com> 
>>>> wrote:
>>>>
>>>>> Joe,
>>>>>
>>>>> I'm not sure if such API clarifications are suitable for update 
>>>>> releases. It hasn't been too regular in the past since we don't 
>>>>> re-publish the API docs post GA of the major release family. I do 
>>>>> understand however that docs may be republished when 8u20 is 
>>>>> released.  8030848 was a corner case issue that came up earlier in 
>>>>> the week. It involved code change and some implementation 
>>>>> clarification via API documentation.
>>>>>
>>>>> I'm cc'ing Joe Darcy here. It would be good to get guidance from 
>>>>> him around suitability of including 8032392 in an update release. 
>>>>> 8032392 appears to be a clarification of spec and I'm checking to 
>>>>> see if that's suitable for an update release.
>>>>>
>>>>> regards,
>>>>> Sean.
>>>>>
>>>>> On 24/01/2014 22:33, huizhe wang wrote:
>>>>>> Hi,
>>>>>>
>>>>>> I'd like to backport the following change to 8u20. This is a 
>>>>>> javadoc enhancement.
>>>>>>
>>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8032392
>>>>>> Changeset: http://hg.openjdk.java.net/jdk9/dev/jaxp/rev/0ee04ce26358
>>>>>> Review thread: 
>>>>>> http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-January/024661.html
>>>>>>
>>>>>> Thanks,
>>>>>> Joe
>>
>