From b26c6df1a13ba4ab28d9099fc31cdfc8d2d0c6e9 Mon Sep 17 00:00:00 2001
From: Andy Clement <aclement@vmware.com>
Date: Sat, 16 Aug 2008 01:57:13 +0000
Subject: [PATCH] fixing TODOs

---
 .../springframework/expression/EvaluationException.java    | 1 -
 .../org/springframework/expression/OperatorOverloader.java | 7 +++----
 2 files changed, 3 insertions(+), 5 deletions(-)

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.