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 2008/06/05 09:58:34 UTC

svn commit: r663495 - /geronimo/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/VariableInterpolator.java

Author: jdillon
Date: Thu Jun  5 00:58:34 2008
New Revision: 663495

URL: http://svn.apache.org/viewvc?rev=663495&view=rev
Log:
Update javadocs

Modified:
    geronimo/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/VariableInterpolator.java

Modified: geronimo/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/VariableInterpolator.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/VariableInterpolator.java?rev=663495&r1=663494&r2=663495&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/VariableInterpolator.java (original)
+++ geronimo/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/VariableInterpolator.java Thu Jun  5 00:58:34 2008
@@ -36,7 +36,10 @@
 import java.util.Set;
 
 /**
- * Provides interpolation for shell variables.
+ * Provides interpolation for shell variables using Jexl.
+ * 
+ * Still using Jexl here for now, since it can handle expression like <tt>${env.TERM}</tt>
+ * (where <tt>env</tt> is a variable bound to a map, ...).
  * 
  * @version $Rev$ $Date$
  */
@@ -153,8 +156,6 @@
 
             public Object getValue(final String s) {
                 try {
-                    // Still using Jexl here for now, since it can handle expression like ${env.TERM}
-                    // (where "env" is a variable bound to a map, ...)
                     Expression expr = ExpressionFactory.createExpression(s);
                     expr.addPreResolver(resolver);