On 21/06/2010, at 7:26 PM, Adam Heath wrote: > 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.
Gotcha, the double loading still applies though, why not recheck the cache before proceeding? >> >> 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