diff --git a/org.springframework.expression/src/main/java/org/springframework/expression/EvaluationException.java b/org.springframework.expression/src/main/java/org/springframework/expression/EvaluationException.java index 264e0f7a0d..4bbf094d2f 100644 --- a/org.springframework.expression/src/main/java/org/springframework/expression/EvaluationException.java +++ b/org.springframework.expression/src/main/java/org/springframework/expression/EvaluationException.java @@ -15,7 +15,6 @@ */ package org.springframework.expression; -// TODO (asc) This class needs a better name? I might have used EvaluationException if it wasn't also used for parsing issues /** * Base class for exceptions occurring during expression parsing and evaluation. * diff --git a/org.springframework.expression/src/main/java/org/springframework/expression/OperatorOverloader.java b/org.springframework.expression/src/main/java/org/springframework/expression/OperatorOverloader.java index 022cdc38f1..2d40bc2668 100644 --- a/org.springframework.expression/src/main/java/org/springframework/expression/OperatorOverloader.java +++ b/org.springframework.expression/src/main/java/org/springframework/expression/OperatorOverloader.java @@ -1,6 +1,5 @@ package org.springframework.expression; - /** * By default the mathematical operators {@link Operation} support simple types like numbers. By providing an * implementation of OperatorOverloader, a user of the expression language can support these operations on other types. @@ -9,9 +8,9 @@ package org.springframework.expression; */ public interface OperatorOverloader { - // TODO (asc) does this need a better type name? - // TODO (asc) needs some testing! - + // TODO does type OperatorOverloader need a better name? + // TODO Operator overloading needs some testing! + /** * Return true if the operator overloader supports the specified operation between the two operands and so should be * invoked to handle it.