Scott Gray wrote:
> Won't that result in the bundle being loaded twice if another thread is 
> waiting for entry to create the same bundle?

no.  read it again.  the bundle is not reassigned between the first
null check, and the second not null check.  there is no way for the
second if block to ever actually do anything.

> 
> Regards
> Scott
> 
> On 21/06/2010, at 7:09 PM, doo...@apache.org wrote:
> 
>> Author: doogie
>> Date: Mon Jun 21 07:09:13 2010
>> New Revision: 956472
>>
>> URL: http://svn.apache.org/viewvc?rev=956472&view=rev
>> Log:
>> Remove useless check for non-null bundle, immediately after a null
>> check, in UtilResourceBundle.getBundle().
>>
>> Modified:
>>    ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilProperties.java
>>
>> Modified: 
>> ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilProperties.java
>> URL: 
>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilProperties.java?rev=956472&r1=956471&r2=956472&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilProperties.java 
>> (original)
>> +++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilProperties.java 
>> Mon Jun 21 07:09:13 2010
>> @@ -896,9 +896,6 @@ public class UtilProperties implements S
>>             UtilResourceBundle bundle = bundleCache.get(resourceName);
>>             if (bundle == null) {
>>                 synchronized (bundleCache) {
>> -                    if (bundle != null) {
>> -                        return bundle;
>> -                    }
>>                     double startTime = System.currentTimeMillis();
>>                     FastList<Locale> candidateLocales = (FastList<Locale>) 
>> getCandidateLocales(locale);
>>                     UtilResourceBundle parentBundle = null;
>>
>>
> 

Reply via email to