You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by si...@apache.org on 2011/05/18 11:30:43 UTC

svn commit: r1124133 - /incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/enhance/ExpressionCompiler.java

Author: simonetripodi
Date: Wed May 18 09:30:43 2011
New Revision: 1124133

URL: http://svn.apache.org/viewvc?rev=1124133&view=rev
Log:
removed redundant null check

Modified:
    incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/enhance/ExpressionCompiler.java

Modified: incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/enhance/ExpressionCompiler.java
URL: http://svn.apache.org/viewvc/incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/enhance/ExpressionCompiler.java?rev=1124133&r1=1124132&r2=1124133&view=diff
==============================================================================
--- incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/enhance/ExpressionCompiler.java (original)
+++ incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/enhance/ExpressionCompiler.java Wed May 18 09:30:43 2011
@@ -519,11 +519,8 @@ public class ExpressionCompiler
 
             getBody = generateOgnlGetter( newClass, valueGetter, nodeMember );
 
-            if ( setExpression == null )
-            {
-                setExpression = CtNewMethod.setter( "setExpression", nodeMember );
-                newClass.addMethod( setExpression );
-            }
+            setExpression = CtNewMethod.setter( "setExpression", nodeMember );
+            newClass.addMethod( setExpression );
         }
 
         try