Return-Path: Delivered-To: apmail-commons-dev-archive@www.apache.org Received: (qmail 93468 invoked from network); 2 Sep 2009 15:09:00 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 2 Sep 2009 15:09:00 -0000 Received: (qmail 451 invoked by uid 500); 2 Sep 2009 15:08:59 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 317 invoked by uid 500); 2 Sep 2009 15:08:59 -0000 Mailing-List: contact dev-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Commons Developers List" Delivered-To: mailing list dev@commons.apache.org Received: (qmail 307 invoked by uid 99); 2 Sep 2009 15:08:59 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 02 Sep 2009 15:08:59 +0000 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of lists@nabble.com designates 216.139.236.158 as permitted sender) Received: from [216.139.236.158] (HELO kuber.nabble.com) (216.139.236.158) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 02 Sep 2009 15:08:50 +0000 Received: from isper.nabble.com ([192.168.236.156]) by kuber.nabble.com with esmtp (Exim 4.63) (envelope-from ) id 1MirRm-00084p-65 for dev@commons.apache.org; Wed, 02 Sep 2009 08:08:30 -0700 Message-ID: <25259429.post@talk.nabble.com> Date: Wed, 2 Sep 2009 08:08:30 -0700 (PDT) From: Henrib To: dev@commons.apache.org Subject: Re: svn commit: r810445 - in /commons/proper/jexl/branches/2.0/src: main/java/org/apache/commons/jexl/ main/java/org/apache/commons/jexl/parser/ test/java/org/apache/commons/jexl/ In-Reply-To: <25aac9fc0909020331g391d546ek9e55953a717e94dd@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Nabble-From: hbiestro@gmail.com References: <20090902102303.EBA3723888E9@eris.apache.org> <25aac9fc0909020331g391d546ek9e55953a717e94dd@mail.gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org sebb-2-2 wrote: > > > Seems like a complicated way of fixing the problem; I would have > thought it was simpler to ensure that multiple statements are > separated by ";" instead. > > i.e. a script consists of > > Statement ( ";" Statement )* > > So long as "Statement" includes a null statement, this would allow for > a trailing ";". > > But I don't know JavaCC all that well. > Did you try fixing the grammar using what you suggest ? If you actually have implemented it in a simpler way - versus thought about one - please, share the code so we can all learn; I've tried many solutions and did not come up with the current code easily. Besides, the fix forces 2 expressions to be separated by ';' in a statement - which is not equivalent to separate all statements with ';' (';' needed between blocks?). -- View this message in context: http://www.nabble.com/svn-commit%3A-r810445---in--commons-proper-jexl-branches-2.0-src%3A-main-java-org-apache-commons-jexl--main-java-org-apache-commons-jexl-parser--test-java-org-apache-commons-jexl--tp25255016p25259429.html Sent from the Commons - Dev mailing list archive at Nabble.com. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org For additional commands, e-mail: dev-help@commons.apache.org