[jira] [Resolved] (GROOVY-8402) GroovyScriptEngine.ScriptClassLoader applies compilation customizers twice

2017-12-11 Thread Shil Sinha (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8402?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Shil Sinha resolved GROOVY-8402. Resolution: Fixed Fix Version/s: 2.4.14 > GroovyScriptEngine.ScriptClassLoader applies

[jira] [Commented] (GROOVY-8402) GroovyScriptEngine.ScriptClassLoader applies compilation customizers twice

2017-12-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16287171#comment-16287171 ] ASF GitHub Bot commented on GROOVY-8402: Github user asfgit closed the pull request at:

[GitHub] groovy pull request #641: GROOVY-8402: GroovyScriptEngine.ScriptClassLoader ...

2017-12-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/groovy/pull/641 ---

[jira] [Commented] (GROOVY-8412) Support !instanceof flow typing for else case

2017-12-11 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16287165#comment-16287165 ] Paul King commented on GROOVY-8412: --- I remove the code which previously treated instanceof and

[jira] [Commented] (GROOVY-8411) Support instanceof-like flow typing for class literal switch case

2017-12-11 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16287159#comment-16287159 ] Paul King commented on GROOVY-8411: --- This should be possible. It would need to take into account

[jira] [Commented] (GROOVY-8406) DefaultGroovyMethods missing Array support

2017-12-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16286747#comment-16286747 ] ASF GitHub Bot commented on GROOVY-8406: Github user paulk-asert commented on a diff in the pull

[GitHub] groovy pull request #644: GROOVY-8406: Various DefaultGroovyMethods missing ...

2017-12-11 Thread paulk-asert
Github user paulk-asert commented on a diff in the pull request: https://github.com/apache/groovy/pull/644#discussion_r156233058 --- Diff: src/main/org/codehaus/groovy/runtime/DefaultGroovyMethods.java --- @@ -3371,77 +3396,179 @@ public static Number count(byte[] self, Object

[jira] [Commented] (GROOVY-8406) DefaultGroovyMethods missing Array support

2017-12-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16286634#comment-16286634 ] ASF GitHub Bot commented on GROOVY-8406: Github user shils commented on a diff in the pull

[jira] [Created] (GROOVY-8413) Potential issue with indirectImportCheckEnabled in SecureASTCustomizer

2017-12-11 Thread Tim Biggin (JIRA)
Tim Biggin created GROOVY-8413: -- Summary: Potential issue with indirectImportCheckEnabled in SecureASTCustomizer Key: GROOVY-8413 URL: https://issues.apache.org/jira/browse/GROOVY-8413 Project: Groovy

[GitHub] groovy pull request #644: GROOVY-8406: Various DefaultGroovyMethods missing ...

2017-12-11 Thread shils
Github user shils commented on a diff in the pull request: https://github.com/apache/groovy/pull/644#discussion_r156211571 --- Diff: src/main/org/codehaus/groovy/runtime/DefaultGroovyMethods.java --- @@ -3371,77 +3396,179 @@ public static Number count(byte[] self, Object value) {

[jira] [Updated] (GROOVY-8413) Potential issue with indirectImportCheckEnabled in SecureASTCustomizer

2017-12-11 Thread Tim Biggin (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8413?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tim Biggin updated GROOVY-8413: --- Description: I have been attempting to use *SecureASTCustomizer* to secure Groovy scripts, but I've

[jira] [Commented] (GROOVY-8410) Provide a common cache

2017-12-11 Thread Jochen Theodorou (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16286364#comment-16286364 ] Jochen Theodorou commented on GROOVY-8410: -- I think CommonCache is doing to much. You try to

[GitHub] groovy pull request #:

2017-12-11 Thread blackdrag
Github user blackdrag commented on the pull request: https://github.com/apache/groovy/commit/29727ea0c0318304ded181aebfa8cce661d34c56#commitcomment-26193580 In src/main/org/codehaus/groovy/runtime/memoize/CommonCache.java: In

[jira] [Created] (GROOVY-8412) Support !instanceof flow typing for else case

2017-12-11 Thread Eric Milles (JIRA)
Eric Milles created GROOVY-8412: --- Summary: Support !instanceof flow typing for else case Key: GROOVY-8412 URL: https://issues.apache.org/jira/browse/GROOVY-8412 Project: Groovy Issue Type:

[jira] [Created] (GROOVY-8411) Support instanceof-like flow typing for class literal switch case

2017-12-11 Thread Eric Milles (JIRA)
Eric Milles created GROOVY-8411: --- Summary: Support instanceof-like flow typing for class literal switch case Key: GROOVY-8411 URL: https://issues.apache.org/jira/browse/GROOVY-8411 Project: Groovy

[jira] [Comment Edited] (GROOVY-8410) Provide a common cache

2017-12-11 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16285480#comment-16285480 ] Daniel Sun edited comment on GROOVY-8410 at 12/11/17 3:40 PM: -- Hi Jochen,

[jira] [Commented] (GROOVY-8406) DefaultGroovyMethods missing Array support

2017-12-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16286030#comment-16286030 ] ASF GitHub Bot commented on GROOVY-8406: GitHub user paulk-asert opened a pull request:

[GitHub] groovy pull request #644: GROOVY-8406: Various DefaultGroovyMethods missing ...

2017-12-11 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/groovy/pull/644 GROOVY-8406: Various DefaultGroovyMethods missing Array variants resu… …lting in no type inference You can merge this pull request into a Git repository by running: $ git pull

[jira] [Assigned] (GROOVY-8406) DefaultGroovyMethods missing Array support

2017-12-11 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8406?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Paul King reassigned GROOVY-8406: - Assignee: Paul King > DefaultGroovyMethods missing Array support >

[jira] [Updated] (GROOVY-8410) Provide a common cache

2017-12-11 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8410?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Sun updated GROOVY-8410: --- Description: There are many cache implementation in Groovy, many of them are duplicated(e.g. cache

[jira] [Comment Edited] (GROOVY-8410) Provide a common cache

2017-12-11 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16285485#comment-16285485 ] Daniel Sun edited comment on GROOVY-8410 at 12/11/17 3:08 PM: -- Here is the

[jira] [Updated] (GROOVY-8410) Provide a common cache

2017-12-11 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8410?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Sun updated GROOVY-8410: --- Description: There are many cache implementation in Groovy, many of them are duplicated(e.g. cache

[jira] [Commented] (GROOVY-8241) SAM parameter type inference for explicit parameter

2017-12-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16285869#comment-16285869 ] ASF GitHub Bot commented on GROOVY-8241: GitHub user avafanasiev opened a pull request:

[GitHub] groovy pull request #643: GROOVY-8241 SAM parameter type inference for expli...

2017-12-11 Thread avafanasiev
GitHub user avafanasiev opened a pull request: https://github.com/apache/groovy/pull/643 GROOVY-8241 SAM parameter type inference for explicit parameter It should fix GROOVY-8241, GROOVY-7061, GROOVY-8317. You can merge this pull request into a Git repository by running: $ git

[jira] [Updated] (GROOVY-8410) Provide a common cache

2017-12-11 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8410?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Sun updated GROOVY-8410: --- Description: There are many cache implementation in Groovy, many of them are duplicated(e.g. cache