RFR (JAXP) 8028822 : Error in the documentation for newFactory method of the javax.xml.stream factories
daniel.fuchs at oracle.com
Tue Nov 26 22:23:26 UTC 2013
On 11/26/13 11:13 PM, huizhe wang wrote:
> On 11/26/2013 1:37 PM, Alan Bateman wrote:
>> On 26/11/2013 18:38, huizhe wang wrote:
>>> Hi all,
>>> Here's revised webrev, as Alan suggested, including the implementation.
>> So is this handling the null case correctly? It is spec'ed to use the
>> TCCL but it looks like it's going through to ServiceLoader.load where
>> null means the system class loader. Maybe this needs to wait for tests
>> as that it really the only way to check this.
> Ah, I missed that. I took it for granted that since
> ServiceLoader.load(service) uses TCCL, load(service, null) would use
> TCCL. I've updated the webrev now:
This looks good.
> I've re-submitted a JPRT build, will send that to JCK.
More information about the core-libs-dev