Parser version

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

Parser version

Russel Winder-3
Is the ANTLR-2 parser really still the default one for Git master/HEAD?
 
--
Russel.
===========================================
Dr Russel Winder      t: +44 20 7585 2200
41 Buckmaster Road    m: +44 7770 465 077
London SW11 1EN, UK   w: www.russel.org.uk

signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: Parser version

Daniel.Sun
Hi Russel,

      ANTLR-4 parser(i.e. the Parrot parser) is the default one for master,
and ANTLR-2 parser is default for other branches.

Cheers,
Daniel.Sun




--
Sent from: http://groovy.329449.n5.nabble.com/Groovy-Dev-f372993.html
Daniel Sun
Apache Groovy committer

Blog: http://blog.sunlan.me
Twitter: @daniel_sun
Reply | Threaded
Open this post in threaded view
|

Re: Parser version

Russel Winder-3
On Thu, 2018-02-08 at 10:13 -0700, Daniel.Sun wrote:
> Hi Russel,
>
>       ANTLR-4 parser(i.e. the Parrot parser) is the default one for
> master,
> and ANTLR-2 parser is default for other branches.
>
> Cheers,
> Daniel.Sun

Strange. I am building master and there is definitely an ANTLR2
generation phase of the build.

--
Russel.
===========================================
Dr Russel Winder      t: +44 20 7585 2200
41 Buckmaster Road    m: +44 7770 465 077
London SW11 1EN, UK   w: www.russel.org.uk

signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: Parser version

paulk_asert
Currently we still include the antlr2 parser in Groovy 3 but it is dormant unless you turn off the new parser with the flag -Dgroovy.antlr4=false. We've really only left it there as a porting aid - in case there ends up being some code that doesn't work with the new parser, there'll be an interim approach until the code is "ported" to the new parser. When we get around to thinking about 3.1, we'll probably remove it altogether.

Cheers, Paul.

On Sat, Feb 10, 2018 at 2:03 AM, Russel Winder <[hidden email]> wrote:
On Thu, 2018-02-08 at 10:13 -0700, Daniel.Sun wrote:
> Hi Russel,
>
>       ANTLR-4 parser(i.e. the Parrot parser) is the default one for
> master,
> and ANTLR-2 parser is default for other branches.
>
> Cheers,
> Daniel.Sun

Strange. I am building master and there is definitely an ANTLR2
generation phase of the build.

--
Russel.
===========================================
Dr Russel Winder      t: <a href="tel:%2B44%2020%207585%202200" value="+442075852200">+44 20 7585 2200
41 Buckmaster Road    m: <a href="tel:%2B44%207770%20465%20077" value="+447770465077">+44 7770 465 077
London SW11 1EN, UK   w: www.russel.org.uk