commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From d...@apache.org
Subject svn commit: r587677 - /commons/proper/jexl/branches/2.0/src/java/org/apache/commons/jexl/parser/Parser.jjt
Date Tue, 23 Oct 2007 22:12:57 GMT
Author: dion
Date: Tue Oct 23 15:12:56 2007
New Revision: 587677

URL: http://svn.apache.org/viewvc?rev=587677&view=rev
Log:
Add a comment about grammar issues
Remove quotes around string literal image

Modified:
    commons/proper/jexl/branches/2.0/src/java/org/apache/commons/jexl/parser/Parser.jjt

Modified: commons/proper/jexl/branches/2.0/src/java/org/apache/commons/jexl/parser/Parser.jjt
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0/src/java/org/apache/commons/jexl/parser/Parser.jjt?rev=587677&r1=587676&r2=587677&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0/src/java/org/apache/commons/jexl/parser/Parser.jjt (original)
+++ commons/proper/jexl/branches/2.0/src/java/org/apache/commons/jexl/parser/Parser.jjt Tue
Oct 23 15:12:56 2007
@@ -152,6 +152,11 @@
   ConditionalOrExpression()
 }
 
+/*
+ * WTF? How is the LHS of the assignment a 'PrimaryExpression'?
+ * This includes Literal, Null, String literal etc...
+ */
+
 void Assignment() #Assignment(2) :
 {}
 {
@@ -357,7 +362,7 @@
 {
  (
   t=<STRING_LITERAL>
-  { jjtThis.image = t.image; }
+  { jjtThis.image = t.image.substring(1, t.image.length() -1); }
  )
 }
 



Mime
View raw message