Re: [PROGRESS REPORT 20161004]groovy-parser

2016-10-09 Thread Guillaume Laforge
I agree we should also try the new parser on a handful of other popular projects, as it'd ensure the parser can parse even more of Groovy that is in the wild. Le dimanche 9 octobre 2016, Thibault Kruse a écrit : > Should the new parser also pass tests for the most-used frameworks > before being

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-09 Thread daniel_sun
this email, your message will be added to the discussion below: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735942.html To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-09 Thread Jochen Theodorou
On 07.10.2016 18:09, daniel_sun wrote: Jochen, you can see the test result at: http://ci.groovy-lang.org/viewType.html?buildTypeId=Groovy_Antlr4grammar thanks... but there are 3 tests failing. Are those the test you have a local fix for? bye Jochen

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-08 Thread daniel_sun
r. > > bye Jochen If you reply to this email, your message will be added to the discussion below: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735940.html To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, cli

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-08 Thread Thibault Kruse
Should the new parser also pass tests for the most-used frameworks before being merged? Grails, Gradle, Spock and Geb come to my mind. On Fri, Oct 7, 2016 at 8:40 PM, Jochen Theodorou wrote: > > > On 07.10.2016 13:35, daniel_sun wrote: >> >> I see. >> >> Feel free to merge the new parser back whe

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-07 Thread daniel_sun
dded to the discussion below: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735891.html To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click here<http://groovy.329449.n5.nabble.com/template/NamlServlet.jtp?macro=unsubs

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-07 Thread daniel_sun
d at least 3(?) this year. bye Jochen If you reply to this email, your message will be added to the discussion below: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735891.html To unsubscribe from [PROGRESS REPORT 20161004]groovy-parser, click h

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-07 Thread Jochen Theodorou
On 07.10.2016 13:35, daniel_sun wrote: I see. Feel free to merge the new parser back when you think it is mature and ready to go home ;) Currently The parser is lack of some syntax checks(e.g. continue should only be used in loop) and friendly messages for developers. Before that I would li

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-07 Thread daniel_sun
arser. WE can still think about porting or not, once we have it merged back. bye Jochen If you reply to this email, your message will be added to the discussion below: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735888.

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-07 Thread Jochen Theodorou
On 07.10.2016 08:52, daniel_sun wrote: How about providing a jar file containing the new parser, which is compiled by Java8. When developers use Java8, they can try the new parser by turning on the switch. I think for the time being we could make the new parser just a subproject that depends

Re: [PROGRESS REPORT 20161004]groovy-parser

2016-10-07 Thread Paul King
On Fri, Oct 7, 2016 at 8:16 PM, Thibault Kruse wrote: > On Tue, Oct 4, 2016 at 1:22 AM, daniel_sun wrote: >> 2) >> subprojects/groovy-groovysh/src/main/groovy/org/codehaus/groovy/tools/shell/commands/DocCommand.groovy(TODO >> confirm: https://github.com/danielsun1106/groovy-parser/issues/5); > >

Re: [PROGRESS REPORT 20161004]groovy-parser

2016-10-07 Thread Thibault Kruse
On Tue, Oct 4, 2016 at 1:22 AM, daniel_sun wrote: > 2) > subprojects/groovy-groovysh/src/main/groovy/org/codehaus/groovy/tools/shell/commands/DocCommand.groovy(TODO > confirm: https://github.com/danielsun1106/groovy-parser/issues/5); So the current Groovy parser could handle this case?

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-06 Thread daniel_sun
t;> >> PS: The new parser requires Java8 because streams and lambda expression >> are >> used. >> >> >> Cheers, >> Daniel.Sun >> >> >> 发件人: Graeme Rocher-2 [via Groovy] >> 发送时间: 2016年10月6日 18:27 >> 收件人: daniel_sun >> 主题: R

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-06 Thread Paul King
;> >> PS: The new parser requires Java8 because streams and lambda expression >> are >> used. >> >> >> Cheers, >> Daniel.Sun >> >> >> 发件人: Graeme Rocher-2 [via Groovy] >> 发送时间: 2016年10月6日 18:27 >> 收件人: da

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-06 Thread daniel_sun
t; > 发件人: Graeme Rocher-2 [via Groovy] > 发送时间: 2016年10月6日 18:27 > 收件人: daniel_sun > 主题: Re: 答复: [PROGRESS REPORT 20161004]groovy-parser > > Hi Daniel, > > Yes I have seen that before, it happens when Travis runs out of memory > since each Travis

Re: 答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-06 Thread Graeme Rocher
because streams and lambda expression are > used. > > > Cheers, > Daniel.Sun > > > 发件人: Graeme Rocher-2 [via Groovy] > 发送时间: 2016年10月6日 18:27 > 收件人: daniel_sun > 主题: Re: 答复: [PROGRESS REPORT 20161004]groovy-parser > > Hi Daniel, &g

答复: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-06 Thread daniel_sun
hanks, > Daniel.Sun > ________ > 发件人: Graeme Rocher-2 [via Groovy] > 发送时间: 2016年10月6日 17:29 > 收件人: daniel_sun > 主题: Re: [PROGRESS REPORT 20161004]groovy-parser > > That is awesome, keep up the good work. > > Cheers > > On Wed, Oct 5, 2

Re: 答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-06 Thread Graeme Rocher
_ > 发件人: Graeme Rocher-2 [via Groovy] > 发送时间: 2016年10月6日 17:29 > 收件人: daniel_sun > 主题: Re: [PROGRESS REPORT 20161004]groovy-parser > > That is awesome, keep up the good work. > > Cheers > > On Wed, Oct 5, 2016 at 5:57 AM, daniel_sun <[hidden email]>

答复: [PROGRESS REPORT 20161004]groovy-parser

2016-10-06 Thread daniel_sun
10月6日 17:29 收件人: daniel_sun 主题: Re: [PROGRESS REPORT 20161004]groovy-parser That is awesome, keep up the good work. Cheers On Wed, Oct 5, 2016 at 5:57 AM, daniel_sun <[hidden email]> wrote: > Thanks :) > > > > -- > View this message in context: > http://groovy.32

Re: [PROGRESS REPORT 20161004]groovy-parser

2016-10-06 Thread Graeme Rocher
That is awesome, keep up the good work. Cheers On Wed, Oct 5, 2016 at 5:57 AM, daniel_sun wrote: > Thanks :) > > > > -- > View this message in context: > http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735778.html > Sent from the

Re: [PROGRESS REPORT 20161004]groovy-parser

2016-10-04 Thread daniel_sun
I'm trying to set the new parser as the default to build and test, *13396 tests completed, 6 failed, 9 skipped* https://travis-ci.org/danielsun1106/groovy -- View this message in context: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735777.html

RE: [PROGRESS REPORT 20161004]groovy-parser

2016-10-04 Thread daniel_sun
Thanks :) -- View this message in context: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725p5735778.html Sent from the Groovy Dev mailing list archive at Nabble.com.

RE: [PROGRESS REPORT 20161004]groovy-parser

2016-10-04 Thread atuzov .
; > At last, I want to thank *Jochen Theodorou* and *Paul King*, who > provide me many suggestion and help while developping the new parser. > > Cheers, > Daniel.Sun > > > > -- > View this message in context: http://groovy.329449.n5. > nabble.com/PROGRESS-REPORT

RE: [PROGRESS REPORT 20161004]groovy-parser

2016-10-04 Thread Miles, Chris (MFT)
Excellent, well done! From: daniel_sun [realblue...@hotmail.com] Sent: 03 October 2016 17:22 To: d...@groovy.incubator.apache.org Subject: [PROGRESS REPORT 20161004]groovy-parser Hi List, Currently the new parser can parse almost all groovy source

[PROGRESS REPORT 20161004]groovy-parser

2016-10-03 Thread daniel_sun
, Daniel.Sun -- View this message in context: http://groovy.329449.n5.nabble.com/PROGRESS-REPORT-20161004-groovy-parser-tp5735725.html Sent from the Groovy Dev mailing list archive at Nabble.com.