Won't that result in the bundle being loaded twice if another thread is waiting for entry to create the same bundle?
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; > >
smime.p7s
Description: S/MIME cryptographic signature