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

Posted to scm@geronimo.apache.org by jd...@apache.org on 2007/09/11 04:20:45 UTC

svn commit: r574435 - /geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/expression/JexlExpressionEvaluator.java

Author: jdillon
Date: Mon Sep 10 19:20:44 2007
New Revision: 574435

URL: http://svn.apache.org/viewvc?rev=574435&view=rev
Log:
wee comment

Modified:
    geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/expression/JexlExpressionEvaluator.java

Modified: geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/expression/JexlExpressionEvaluator.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/expression/JexlExpressionEvaluator.java?rev=574435&r1=574434&r2=574435&view=diff
==============================================================================
--- geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/expression/JexlExpressionEvaluator.java (original)
+++ geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/expression/JexlExpressionEvaluator.java Mon Sep 10 19:20:44 2007
@@ -33,6 +33,12 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+//
+// TODO: Consider using BeanShell for this instead, its about the same size dependency
+//       can probably do about the same thing we want here?  And the 'script' command
+//       can also use it
+//
+
 /**
  * A very simple expression evalutator using Jexl to do the hard work.
  *