RFR: javax.xml.stream: Using ServiceLoader to load JAXP stream factories (7169894: JAXP Plugability Layer: using service loader)

[Mandy Chung]

On 1/9/2013 1:13 PM, Daniel Fuchs wrote:
> Hi Mandy,
>
> Please find clarifications in line:
>
> On 1/9/13 9:30 PM, Mandy Chung wrote:
>>
>>    Since there is a behavioral change, the following statements are 
>> not true
>> any more and I think they can be removed.
>>
>>  166    * No changes in behavior are defined by this replacement 
>> method relative
>>  167    * to the deprecated method.
> The behavior change is the same in both the deprecated & the new 
> method - since
> it occurred in the FactoryFinder which both methods are calling, so I
> think this statement is still true.

I interpreted it differently comparing the old implementation of the 
deprecated newInstance method with the new implementation of newFactory 
method. My interpretation may be wrong and so will leave it to Joe to 
comment.  I'm fine with what you currently have.

>>
>> The classLoader parameter passed to the newFactory(String, 
>> ClassLoader) method
>> is actually ignored.  It might be good to clarify that since the spec 
>> is updated
>> anyway.
>
> I think it's a bug. I'm intending to log it as a separate issue. FWIW 
> in the other factories
> (in other packages) the classloader is not ignored.

In that case, please file a bug to track this.

Thanks
Mandy