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()) {