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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
