Code Review for 6998860

Mandy Chung mandy.chung at
Tue Dec 7 17:20:35 PST 2010

  Hi Sean,

On 12/7/10 12:47 PM, Sean Mullan wrote:
> Hi Mandy,
> Could I get a code review for 6998860:
Is Providers.getSunProvider() specified to create a new instance of the 
provider every time it's called?   I would assume that the runtime 
should create only one single instance of the Sun provider.

In the comment of the 
method, this method is called in two places.

     // Return to Sun provider or its backup.
     // This method should only be called by
     // and
     public static Provider getSunProvider() {

Is it correct to change this method to cache and return a singleton object?


More information about the security-dev mailing list