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

Posted to commits@commons.apache.org by di...@apache.org on 2007/10/24 00:12:57 UTC

svn commit: r587677 - /commons/proper/jexl/branches/2.0/src/java/org/apache/commons/jexl/parser/Parser.jjt

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); }
  )
 }