You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by do...@apache.org on 2010/04/04 20:52:27 UTC

svn commit: r930733 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java

Author: doogie
Date: Sun Apr  4 18:52:26 2010
New Revision: 930733

URL: http://svn.apache.org/viewvc?rev=930733&view=rev
Log:
There is really no need to protect hint with a synchronized block.

Modified:
    ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java

Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java?rev=930733&r1=930732&r2=930733&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/string/FlexibleStringExpander.java Sun Apr  4 18:52:26 2010
@@ -396,9 +396,7 @@ public abstract class FlexibleStringExpa
             buffer.append(obj);
         }
         if (buffer.length() > this.hint) {
-            synchronized(this) {
-                this.hint = buffer.length();
-            }
+            this.hint = buffer.length();
         }
         return buffer.toString();
     }