[8u20] RFA: 8032392 : Spec: javax.xml.stream.XMLEventFactory/XMLOutputFactory/XMLInputFactory.newFactory(String, ClassLoader) referring to ServiceLoader.load(Class, ClassLoader)
joe.darcy at oracle.com
Mon Jan 27 23:50:14 UTC 2014
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.
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).
>> On Jan 25, 2014, at 12:21 PM, Seán Coffey <sean.coffey at oracle.com> wrote:
>>> 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.
>>> On 24/01/2014 22:33, huizhe wang wrote:
>>>> 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
More information about the jdk8u-dev