[GitHub] [groovy] Goooler opened a new pull request, #1787: Migrate JavaPluginConvention to JavaPluginExtension

2022-09-27 Thread GitBox
Goooler opened a new pull request, #1787: URL: https://github.com/apache/groovy/pull/1787 `JavaPluginConvention` has been deprecated, and will be removed in Gradle 8, we should migrate to `JavaPluginExtension`. https://docs.gradle.org/current/userguide/upgrading_version_7.html#java_

[jira] [Commented] (GROOVY-10772) Possible memory leak, CacheableCallSite retains objects across invocations

2022-09-27 Thread Kyle Moore (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10772?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610329#comment-17610329 ] Kyle Moore commented on GROOVY-10772: - We can see that the value of sysprop {{groov

[jira] [Commented] (GROOVY-10772) Possible memory leak, CacheableCallSite retains objects across invocations

2022-09-27 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10772?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610318#comment-17610318 ] Eric Milles commented on GROOVY-10772: -- I think the threshold for call site cache

[jira] [Updated] (GROOVY-10772) Possible memory leak, CacheableCallSite retains objects across invocations

2022-09-27 Thread Sterling Greene (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10772?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sterling Greene updated GROOVY-10772: - Affects Version/s: 4.0.5 4.0.3 > Possible memory leak, Cacheable

[jira] [Commented] (GROOVY-10772) Possible memory leak, CacheableCallSite retains objects across invocations

2022-09-27 Thread Sterling Greene (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10772?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610303#comment-17610303 ] Sterling Greene commented on GROOVY-10772: -- I tried my reproducer with 4.0.0-4

[jira] [Updated] (GROOVY-10772) Possible memory leak, CacheableCallSite retains objects across invocations

2022-09-27 Thread Sterling Greene (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10772?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sterling Greene updated GROOVY-10772: - Description: We're seeing this in a test with Gradle + Groovy 4, so it's hard to reprod

[jira] [Updated] (GROOVY-10772) Possible memory leak, CacheableCallSite retains objects across invocations

2022-09-27 Thread Sterling Greene (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10772?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sterling Greene updated GROOVY-10772: - Description: We're seeing this in a test with Gradle + Groovy 4, so it's hard to reprod

[jira] [Created] (GROOVY-10772) Possible memory leak, CacheableCallSite retains objects across invocations

2022-09-27 Thread Sterling Greene (Jira)
Sterling Greene created GROOVY-10772: Summary: Possible memory leak, CacheableCallSite retains objects across invocations Key: GROOVY-10772 URL: https://issues.apache.org/jira/browse/GROOVY-10772

[jira] [Resolved] (GROOVY-10771) Type-checking extension scripts should import AST classes

2022-09-27 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Milles resolved GROOVY-10771. -- Fix Version/s: 5.0.0-alpha-1 Resolution: Fixed https://github.com/apache/groovy/commit

[jira] [Commented] (GROOVY-10682) Provide eachWithIndex for primitive arrays

2022-09-27 Thread Paul King (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10682?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610278#comment-17610278 ] Paul King commented on GROOVY-10682: Some general comments: * I wouldn't assume we

[jira] [Comment Edited] (GROOVY-10771) Type-checking extension scripts should import AST classes

2022-09-27 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610252#comment-17610252 ] Eric Milles edited comment on GROOVY-10771 at 9/27/22 10:07 PM: -

[jira] [Assigned] (GROOVY-10771) Type-checking extension scripts should import AST classes

2022-09-27 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Milles reassigned GROOVY-10771: Assignee: Eric Milles > Type-checking extension scripts should import AST classes >

[jira] [Commented] (GROOVY-10771) Type-checking extension scripts should import AST classes

2022-09-27 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610252#comment-17610252 ] Eric Milles commented on GROOVY-10771: -- Current script imports are as follows (fro

[jira] [Assigned] (GROOVY-10770) Multiple @CompileStatic annotations don't combine extensions

2022-09-27 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10770?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Milles reassigned GROOVY-10770: Assignee: Eric Milles > Multiple @CompileStatic annotations don't combine extensions > -

[jira] [Commented] (GROOVY-10771) Type-checking extension scripts should import AST classes

2022-09-27 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610243#comment-17610243 ] Eric Milles commented on GROOVY-10771: -- You can add any imports you need to the to

[jira] [Comment Edited] (GROOVY-10771) Type-checking extension scripts should import AST classes

2022-09-27 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610241#comment-17610241 ] Eric Milles edited comment on GROOVY-10771 at 9/27/22 9:04 PM: --

[jira] [Comment Edited] (GROOVY-10771) Type-checking extension scripts should import AST classes

2022-09-27 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610241#comment-17610241 ] Eric Milles edited comment on GROOVY-10771 at 9/27/22 9:01 PM: --

[jira] [Commented] (GROOVY-10771) Type-checking extension scripts should import AST classes

2022-09-27 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610241#comment-17610241 ] Eric Milles commented on GROOVY-10771: -- Your type checking script extends from {{

[jira] [Commented] (GROOVY-10771) Type-checking extension scripts should import AST classes

2022-09-27 Thread Christopher Smith (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610216#comment-17610216 ] Christopher Smith commented on GROOVY-10771: I note that in the documentati

[jira] [Created] (GROOVY-10771) Type-checking extension scripts should import AST classes

2022-09-27 Thread Christopher Smith (Jira)
Christopher Smith created GROOVY-10771: -- Summary: Type-checking extension scripts should import AST classes Key: GROOVY-10771 URL: https://issues.apache.org/jira/browse/GROOVY-10771 Project: Groov

[jira] [Created] (GROOVY-10770) Multiple @CompileStatic annotations don't combine extensions

2022-09-27 Thread Christopher Smith (Jira)
Christopher Smith created GROOVY-10770: -- Summary: Multiple @CompileStatic annotations don't combine extensions Key: GROOVY-10770 URL: https://issues.apache.org/jira/browse/GROOVY-10770 Project: G

[jira] [Commented] (GROOVY-10682) Provide eachWithIndex for primitive arrays

2022-09-27 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-10682?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17609985#comment-17609985 ] ASF GitHub Bot commented on GROOVY-10682: - Shadow-Devil commented on code in PR

[GitHub] [groovy] Shadow-Devil commented on a diff in pull request #1785: GROOVY-10682: Provide eachWithIndex for primitive arrays

2022-09-27 Thread GitBox
Shadow-Devil commented on code in PR #1785: URL: https://github.com/apache/groovy/pull/1785#discussion_r981093151 ## src/test/groovy/transform/stc/DefaultGroovyMethodsSTCTest.groovy: ## @@ -37,6 +37,20 @@ class DefaultGroovyMethodsSTCTest extends StaticTypeCheckingTestCase {