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

Seán Coffey sean.coffey at oracle.com
Tue Jan 28 14:51:14 UTC 2014


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
>




More information about the jdk8u-dev mailing list