commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From d...@apache.org
Subject svn commit: r383307 - /jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/parser/ASTSubtractNode.java
Date Sun, 05 Mar 2006 11:46:03 GMT
Author: dion
Date: Sun Mar  5 03:46:00 2006
New Revision: 383307

URL: http://svn.apache.org/viewcvs?rev=383307&view=rev
Log:
Remove old TODO

Modified:
    jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/parser/ASTSubtractNode.java

Modified: jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/parser/ASTSubtractNode.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/parser/ASTSubtractNode.java?rev=383307&r1=383306&r2=383307&view=diff
==============================================================================
--- jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/parser/ASTSubtractNode.java
(original)
+++ jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/parser/ASTSubtractNode.java
Sun Mar  5 03:46:00 2006
@@ -81,15 +81,6 @@
         Long l = Coercion.coerceLong(left);
         Long r = Coercion.coerceLong(right);
 
-        /*
-         *  TODO - this is actually wrong - JSTL says to return a Long
-         *  but we have problems where you have something like 
-         * 
-         *     foo.bar( a - b )
-         * 
-         *  where bar wants an int... 
-         * 
-         */
         return new Long(l.longValue() - r.longValue());
 
      }



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message