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

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

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

adrianc
Author: adrianc
Date: Mon Apr 22 21:56:03 2013
New Revision: 1470732

URL: http://svn.apache.org/r1470732
Log:
Removed unnecessary synchronized keyword. The GenericEntity class is not thread-safe, so synchronizing a single method is pointless.

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=1470732&r1=1470731&r2=1470732&view=diff
==============================================================================
--- ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericEntity.java (original)
+++ ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericEntity.java Mon Apr 22 21:56:03 2013
@@ -403,7 +403,7 @@ public class GenericEntity implements Ma
      * @param value The value to set
      * @param setIfNull Specifies whether or not to set the value if it is null
      */
-    public synchronized Object set(String name, Object value, boolean setIfNull) {
+    public Object set(String name, Object value, boolean setIfNull) {
         assertIsMutable();
         ModelField modelField = getModelEntity().getField(name);
         if (modelField == null) {