[GitHub] [groovy] paulk-asert commented on issue #910: GROOVY-9083: groovyConsole should be updated to understand method ref…

2019-07-15 Thread GitBox
paulk-asert commented on issue #910: GROOVY-9083: groovyConsole should be updated to understand method ref… URL: https://github.com/apache/groovy/pull/910#issuecomment-511629443 Our goal is to get as many of the tools ported across to the new parser before 3.0 GA but some may have to wait

[GitHub] [groovy] danielsun1106 commented on issue #910: GROOVY-9083: groovyConsole should be updated to understand method ref…

2019-07-15 Thread GitBox
danielsun1106 commented on issue #910: GROOVY-9083: groovyConsole should be updated to understand method ref… URL: https://github.com/apache/groovy/pull/910#issuecomment-511610543 @mvmn groovysh does not support new language features because it still bases on the old parser.

[jira] [Comment Edited] (GROOVY-9159) [GEP] Support LINQ, aka GINQ

2019-07-15 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-9159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16885701#comment-16885701 ] Daniel Sun edited comment on GROOVY-9159 at 7/15/19 11:37 PM: -- [~glaforge] 

[jira] [Commented] (GROOVY-9159) [GEP] Support LINQ, aka GINQ

2019-07-15 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-9159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16885701#comment-16885701 ] Daniel Sun commented on GROOVY-9159: [~glaforge]  GINQ related keywords, e.g. {{from}}, {{where}},

[GitHub] [groovy] mvmn edited a comment on issue #910: GROOVY-9083: groovyConsole should be updated to understand method ref…

2019-07-15 Thread GitBox
mvmn edited a comment on issue #910: GROOVY-9083: groovyConsole should be updated to understand method ref… URL: https://github.com/apache/groovy/pull/910#issuecomment-511587368 Why doesn't this work in Groovy Shell though? ``` >

[GitHub] [groovy] mvmn commented on issue #910: GROOVY-9083: groovyConsole should be updated to understand method ref…

2019-07-15 Thread GitBox
mvmn commented on issue #910: GROOVY-9083: groovyConsole should be updated to understand method ref… URL: https://github.com/apache/groovy/pull/910#issuecomment-511587368 Why doesn't this work in Groovy Shell though? ``` >

[GitHub] [groovy] danielsun1106 commented on issue #965: GROOVY-9192: Implement parser error recovery for Parrot parser

2019-07-15 Thread GitBox
danielsun1106 commented on issue #965: GROOVY-9192: Implement parser error recovery for Parrot parser URL: https://github.com/apache/groovy/pull/965#issuecomment-511492180 Well... the built-in parser error recovery of antlr4 seems not to meet the requirement...

[GitHub] [groovy] eric-milles commented on issue #965: GROOVY-9192: Implement parser error recovery for Parrot parser

2019-07-15 Thread GitBox
eric-milles commented on issue #965: GROOVY-9192: Implement parser error recovery for Parrot parser URL: https://github.com/apache/groovy/pull/965#issuecomment-511488720 Before applying these changes, Groovy-Eclipse has 161 errors and 12 failures in the compiler/builder/editor test suite

[jira] [Commented] (GROOVY-8955) VerifyError when AIC with @CS can't be detected as not valid

2019-07-15 Thread Eric Milles (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8955?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16885340#comment-16885340 ] Eric Milles commented on GROOVY-8955: - Is the expectation some compile-time error if the returned

[jira] [Created] (GROOVY-9193) SC: Script method with default arguments throws IncompatibleClassChangeError

2019-07-15 Thread Eric Milles (JIRA)
Eric Milles created GROOVY-9193: --- Summary: SC: Script method with default arguments throws IncompatibleClassChangeError Key: GROOVY-9193 URL: https://issues.apache.org/jira/browse/GROOVY-9193 Project:

[jira] [Commented] (GROOVY-9068) GroovyScriptEngine causes Metaspace OOM

2019-07-15 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-9068?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16885330#comment-16885330 ] Daniel Sun commented on GROOVY-9068: My last comment above should be a feasible workaround, which is

[jira] [Commented] (GROOVY-9068) GroovyScriptEngine causes Metaspace OOM

2019-07-15 Thread Daniel Wehrle (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-9068?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16885293#comment-16885293 ] Daniel Wehrle commented on GROOVY-9068: --- [~jingfei] [~daniel_sun] is there a solution for this

[GitHub] [groovy] paulk-asert commented on issue #968: Fixed the Reference of the metaclass in ClassInfo.java to use a Weak…

2019-07-15 Thread GitBox
paulk-asert commented on issue #968: Fixed the Reference of the metaclass in ClassInfo.java to use a Weak… URL: https://github.com/apache/groovy/pull/968#issuecomment-511408589 @blackdrag Any thoughts? This is an automated

[jira] [Resolved] (GROOVY-8547) Factory pattern side-stepped by CompilerConfiguration/ParserVersion

2019-07-15 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8547?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Paul King resolved GROOVY-8547. --- Resolution: Fixed Assignee: Paul King Fix Version/s: 3.0.0-beta-3 Proposed PR

[GitHub] [groovy] paulk-asert commented on issue #967: GROOVY-8547: remove ParserVersion in favor of CC.setPluginFactory(PPF)

2019-07-15 Thread GitBox
paulk-asert commented on issue #967: GROOVY-8547: remove ParserVersion in favor of CC.setPluginFactory(PPF) URL: https://github.com/apache/groovy/pull/967#issuecomment-511401399 Merged. Thanks! This is an automated message

[GitHub] [groovy] asfgit closed pull request #967: GROOVY-8547: remove ParserVersion in favor of CC.setPluginFactory(PPF)

2019-07-15 Thread GitBox
asfgit closed pull request #967: GROOVY-8547: remove ParserVersion in favor of CC.setPluginFactory(PPF) URL: https://github.com/apache/groovy/pull/967 This is an automated message from the Apache Git Service. To respond to

[jira] [Commented] (GROOVY-9159) [GEP] Support LINQ, aka GINQ

2019-07-15 Thread Guillaume Laforge (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-9159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16885084#comment-16885084 ] Guillaume Laforge commented on GROOVY-9159: --- [~daniel_sun] So if it's a dedicated syntax (with

[jira] [Commented] (GROOVY-9159) [GEP] Support LINQ, aka GINQ

2019-07-15 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-9159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16885074#comment-16885074 ] Daniel Sun commented on GROOVY-9159: [~glaforge] GINQ is much more complicated than what command

[jira] [Commented] (GROOVY-9159) [GEP] Support LINQ, aka GINQ

2019-07-15 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-9159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16885064#comment-16885064 ] Daniel Sun commented on GROOVY-9159: [~lukas.eder] Thanks for your reminder. The order issue you

[GitHub] [groovy] NecOnIce opened a new pull request #968: Fixed the Reference of the metaclass in ClassInfo.java to use a Weak…

2019-07-15 Thread GitBox
NecOnIce opened a new pull request #968: Fixed the Reference of the metaclass in ClassInfo.java to use a Weak… URL: https://github.com/apache/groovy/pull/968 …Reference now, so the Reference is cleaned up nearly immediately after freeing the last strong reference of the metaclass.

[jira] [Comment Edited] (GROOVY-9159) [GEP] Support LINQ, aka GINQ

2019-07-15 Thread Guillaume Laforge (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-9159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16884948#comment-16884948 ] Guillaume Laforge edited comment on GROOVY-9159 at 7/15/19 8:45 AM:

[jira] [Commented] (GROOVY-9159) [GEP] Support LINQ, aka GINQ

2019-07-15 Thread Guillaume Laforge (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-9159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16884948#comment-16884948 ] Guillaume Laforge commented on GROOVY-9159: --- It's probably too early to discuss syntax as you

[jira] [Comment Edited] (GROOVY-9159) [GEP] Support LINQ, aka GINQ

2019-07-15 Thread Lukas Eder (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-9159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16884913#comment-16884913 ] Lukas Eder edited comment on GROOVY-9159 at 7/15/19 7:54 AM: - {quote}limit 5

[jira] [Commented] (GROOVY-9159) [GEP] Support LINQ, aka GINQ

2019-07-15 Thread Lukas Eder (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-9159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16884913#comment-16884913 ] Lukas Eder commented on GROOVY-9159: limit 5 offset 2 select v When switching the order of