On Fri, Oct 7, 2016 at 8:16 PM, Thibault Kruse <tibokr...@googlemail.com> wrote: > On Tue, Oct 4, 2016 at 1:22 AM, daniel_sun <realblue...@hotmail.com> 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?
There was an erroneous syntax error in that file - would have thrown some kind of runtime exception (missing property or something) if that branch was ever hit. Double quotes were changed to single for a non-GString string (perhaps avoiding a codenarc warning?) but the string already contained single quotes. Anyway, I flipped the inner ones to be double quotes. So there is nothing to fix in the new parser. Paul.
