You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jo...@apache.org on 2008/05/03 08:43:39 UTC

svn commit: r653009 - /ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericEntity.java

Author: jonesde
Date: Fri May  2 23:43:39 2008
New Revision: 653009

URL: http://svn.apache.org/viewvc?rev=653009&view=rev
Log:
Removed redundant synchronized on setFields because it is synced on set

Modified:
    ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericEntity.java

Modified: ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericEntity.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericEntity.java?rev=653009&r1=653008&r2=653009&view=diff
==============================================================================
--- ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericEntity.java (original)
+++ ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericEntity.java Fri May  2 23:43:39 2008
@@ -901,7 +901,7 @@
     /** Used by clients to update particular fields in the entity
      * @param keyValuePairs java.util.Map
      */
-    public synchronized void setFields(Map<? extends String, ? extends Object> keyValuePairs) {
+    public void setFields(Map<? extends String, ? extends Object> keyValuePairs) {
         if (keyValuePairs == null) return;
         // this could be implement with Map.putAll, but we'll leave it like this for the extra features it has
         for (Map.Entry<? extends String, ? extends Object> anEntry: keyValuePairs.entrySet()) {