Mailing-List: contact dev-help@groovy.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@groovy.apache.org
MIME-Version: 1.0
Reply-To: paulk@asert.com.au
In-Reply-To: <BC0E4CC6-3D9D-4D18-9420-ED622D1FA6C1@selskabet.org>
References: <29FC6596-94BC-4BE2-A9A9-2FA48680CC08@selskabet.org>
	<CAJ2HCd8wzWqpgkdn6BZ7zZ93842XghLuzuASdc4tOHjzbzQP9g@mail.gmail.com>
	<BC0E4CC6-3D9D-4D18-9420-ED622D1FA6C1@selskabet.org>
Date: Wed, 24 Feb 2016 21:35:34 +1000
Message-ID: 
 <CAMbkE7QRDnSLD4y4_jJ8Lx1u1ewTomvGkcrs3SSRBmHi5v-awg@mail.gmail.com>
Subject: Re: Antlr4-based Groovy parser status update
From: Paul King <paulk@asert.com.au>
To: dev@groovy.apache.org
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Spock for testing is by no means a show stopper!

On Wed, Feb 24, 2016 at 6:46 PM, Jesper Steen M=C3=B8ller
<jesper@selskabet.org> wrote:
>
> On 24. feb. 2016, at 01.21, Guillaume Laforge <glaforge@gmail.com> wrote:
>
> Sounds cool, Jesper, great to hear about your progress on the Antlr4
> grammar!
> Is there a way we can help you? How can we test drive this? Is the build
> working?
>
>
> The build works in that the antlr4 branch can be build and tested with e.=
g.
> =E2=80=9Cgradle -PuseAntlr4=3Dtrue test=E2=80=9D.
>
> However, I can=E2=80=99t seem to actually build a real distribution and u=
se the new
> parser in that (there=E2=80=99s a dynamic switch), without getting some v=
ery strange
> errors.
> I=E2=80=99ve put off chasing that problem until I know I have the missing=
 pieces of
> the grammar and the AST building done right.
>
> The current tests use Spock. Is that out of the question for the main Gro=
ovy
> build? I loooove Spock, but I realise we have to keep the number of
> dependencies down.
>
> -Jesper
>
>
> On Wed, Feb 24, 2016 at 1:14 AM, Jesper Steen M=C3=B8ller <jesper@selskab=
et.org>
> wrote:
>>
>> Hi Groovy-dev
>>
>> I=E2=80=99ve found some time to work on the Antlr4 parser and fixed some=
 minor
>> tasks there:
>>
>> * Fix grammar for several statement sequences, where NL or semicolon was
>> not required (added negative test cases)
>> * Remove non-reentrant static state in ASTBuilder
>> * Added syntax error handling to Antlr 4 parser
>> * Make sure GrammarPredicates work with arbitrarily nested package
>> hierarchies
>>
>> I=E2=80=99m pushing my changes here:
>> https://github.com/jespersm/groovy/commits/antlr4
>>
>> Next up is support for non-ID method invocation and command expressions,
>> and then unary expressions.
>> Also, since the GSoC is a bit dated, traits are also missing.
>>
>> -Jesper
>
>
>
>
> --
> Guillaume Laforge
> Apache Groovy committer & PMC Vice-President
> Product Ninja & Advocate at Restlet
>
> Blog: http://glaforge.appspot.com/
> Social: @glaforge / Google+
>
>