Review Request: loading JCE providers using ServiceLoader
Paul Sandoz
paul.sandoz at oracle.com
Wed Aug 1 11:58:05 PDT 2012
On Aug 1, 2012, at 11:10 AM, Sean Mullan <sean.mullan at oracle.com> wrote:
> On 07/31/2012 12:26 AM, Paul Sandoz wrote:
>> Hi Sean,
>>
>> The patch looks OK to me.
>>
>> You have a redundant synchronised block in
>> Provider#doLoadProviderUsingServiceLoader you might want to remove.
>
> The lock is on the ServiceLoader and I think that's still necessary. There's a ProviderConfig created for each Provider. It's possible there may be more than one thread accessing the ServiceLoader which is currently shared by all Providers.
Doh! yes, the reference is statically held.
Paul.
More information about the jigsaw-dev
mailing list