svn commit: r898318 - /tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java

[ma...@apache.org]

Author: markt
Date: Tue Jan 12 13:33:58 2010
New Revision: 898318

URL: http://svn.apache.org/viewvc?rev=898318&view=rev
Log:
Update the comment. We don't want or need another full EL parser.

Modified:
    tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java

Modified: tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java?rev=898318&r1=898317&r2=898318&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java (original)
+++ tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java Tue Jan 12 13:33:58 2010
@@ -78,11 +78,12 @@
     }
 
     /**
-     * Parse an EL expression string '${...}'
+     * Parse an EL expression string '${...}'. Currently only separates the EL
+     * into functions and everything else.
      * 
-     * @return An ELNode.Nodes representing the EL expression TODO: Currently
-     *         only parsed into functions and text strings. This should be
-     *         rewritten for a full parser.
+     * @return An ELNode.Nodes representing the EL expression
+     * 
+     * TODO: Can this be refactored to use the standard EL implementation?
      */
     private ELNode.Nodes parseEL() {
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org