Re: [PROGRESS REPORT 20161012]groovy-parser

2016-10-12 Thread daniel_sun
BTW, Groovy 2.5.0 can be built and tested via setting the new parser as the default parser. *groovy-parser:* https://github.com/danielsun1106/groovy-parser *groovy fork:* https://github.com/danielsun1106/groovy Cheers, Daniel.Sun -- View this message in context: http://groovy.329449.n5.nabbl

Re: [PROGRESS REPORT 20161012]groovy-parser

2016-10-12 Thread daniel_sun
Hi Sergei, The new parser generates much more accurate position information of AST nodes, which I think will be a good start of Groovy 3. Thanks for your advice again :) Cheers, Daniel.Sun 在 "Sergei Egorov [via Groovy]" ,2016年10月12日 下午8:53写道: Hi Daniel, I'll repeat - IMO we should fi

Re: [PROGRESS REPORT 20161012]groovy-parser

2016-10-12 Thread Sergei Egorov
Hi Daniel, I'll repeat - IMO we should fix the parser, not the MacroGroovy code. MacroGroovy tests just spotted an issue with the source info (line, column) breaking change. However, if we're planning to integrate Antlr4 grammar as a drop-in solution, I think it must be backward compatible. Other'