[GitHub] [groovy] danielsun1106 commented on pull request #1323: JmxMetaMapBuilder.buildOperationMapFrom throw StringIndexOutOfBoundsException

2020-07-24 Thread GitBox
danielsun1106 commented on pull request #1323: URL: https://github.com/apache/groovy/pull/1323#issuecomment-663812931 It would be better to have a test in this PR This is an automated message from the Apache Git Service. To r

[GitHub] [groovy] hdfg159 opened a new pull request #1323: JmxMetaMapBuilder.buildOperationMapFrom throw StringIndexOutOfBoundsException

2020-07-24 Thread GitBox
hdfg159 opened a new pull request #1323: URL: https://github.com/apache/groovy/pull/1323 JmxMetaMapBuilder.buildOperationMapFrom for method "get" or "set" throw StringIndexOutOfBoundsException example: ``` class TestGet { String sb def get(String str) {

[jira] [Created] (GROOVY-9656) ExecutorService as a @Delegate calls the Runnable submit instead of the Callable one

2020-07-24 Thread Tim Yates (Jira)
Tim Yates created GROOVY-9656: - Summary: ExecutorService as a @Delegate calls the Runnable submit instead of the Callable one Key: GROOVY-9656 URL: https://issues.apache.org/jira/browse/GROOVY-9656 Projec

[jira] [Commented] (GROOVY-9655) Groovy 3.0.5 regression with the "with" keyword

2020-07-24 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-9655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17164401#comment-17164401 ] Eric Milles commented on GROOVY-9655: - You can add "owner." or "this." to the variab

[jira] [Updated] (GROOVY-9655) Groovy 3.0.5 regression with the "with" keyword

2020-07-24 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-9655?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Milles updated GROOVY-9655: Affects Version/s: 2.5.13 > Groovy 3.0.5 regression with the "with" keyword >

[jira] [Resolved] (GROOVY-9648) Bad error message when attempting to call a missing constructor

2020-07-24 Thread Paul King (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-9648?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Paul King resolved GROOVY-9648. --- Fix Version/s: 3.0.6 4.0.0-alpha-1 Assignee: Paul King Resolution:

[GitHub] [groovy] paulk-asert commented on pull request #1321: GROOVY-9648: Bad error message when attempting to call a missing cons…

2020-07-24 Thread GitBox
paulk-asert commented on pull request #1321: URL: https://github.com/apache/groovy/pull/1321#issuecomment-663497524 Merged. This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [groovy] paulk-asert closed pull request #1321: GROOVY-9648: Bad error message when attempting to call a missing cons…

2020-07-24 Thread GitBox
paulk-asert closed pull request #1321: URL: https://github.com/apache/groovy/pull/1321 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[jira] [Assigned] (GROOVY-9655) Groovy 3.0.5 regression with the "with" keyword

2020-07-24 Thread Eric Milles (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-9655?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Milles reassigned GROOVY-9655: --- Assignee: Eric Milles > Groovy 3.0.5 regression with the "with" keyword > -

[jira] [Resolved] (GROOVY-9654) should not consider bridge methods when determining whether an overridden method annotated with @Override is valid

2020-07-24 Thread Paul King (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-9654?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Paul King resolved GROOVY-9654. --- Fix Version/s: 3.0.6 2.5.14 4.0.0-alpha-1 Resolution: Fi

[GitHub] [groovy] paulk-asert edited a comment on pull request #1322: GROOVY-9637: Improve the performance of GString

2020-07-24 Thread GitBox
paulk-asert edited a comment on pull request #1322: URL: https://github.com/apache/groovy/pull/1322#issuecomment-663448502 **_Goal_**: improve the performance of GStrings where possible but without foregoing behavior where a value can be dynamically supplied by a closure including lazily e

[GitHub] [groovy] paulk-asert commented on pull request #1292: Joint build Groovy & Grails

2020-07-24 Thread GitBox
paulk-asert commented on pull request #1292: URL: https://github.com/apache/groovy/pull/1292#issuecomment-663452909 Can you try the latest SNAPSHOT after changing to boolean? This is an automated message from the Apache Git S

[GitHub] [groovy] paulk-asert commented on pull request #1322: GROOVY-9637: Improve the performance of GString

2020-07-24 Thread GitBox
paulk-asert commented on pull request #1322: URL: https://github.com/apache/groovy/pull/1322#issuecomment-663448502 **_Goal_**: improve the performance of GStrings where possible but without foregoing behavior where a value can be dynamically supplied by a closure including lazily evaluate

[GitHub] [groovy] paulk-asert edited a comment on pull request #1322: GROOVY-9637: Improve the performance of GString

2020-07-24 Thread GitBox
paulk-asert edited a comment on pull request #1322: URL: https://github.com/apache/groovy/pull/1322#issuecomment-663448502 **_Goal_**: improve the performance of GStrings where possible but without foregoing behavior where a value can be dynamically supplied by a closure including lazily e

[jira] [Commented] (GROOVY-9651) VariableScopeVisitor does not check object expression resp. MethodCallExpression#isImplicitThis returns unhelpful true

2020-07-24 Thread Jira
[ https://issues.apache.org/jira/browse/GROOVY-9651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17164272#comment-17164272 ] Björn Kautler commented on GROOVY-9651: --- Sounds like a design bug to me. :D > Var

[jira] [Updated] (GROOVY-9655) Groovy 3.0.5 regression with the "with" keyword

2020-07-24 Thread Henrik (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-9655?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Henrik updated GROOVY-9655: --- Description: Since GROOVY-7701 was the only issue relating to {{with}} in Groovy 3.0.5, I think that fix mi

[jira] [Created] (GROOVY-9655) Groovy 3.0.5 regression with the "with" keyword

2020-07-24 Thread Henrik (Jira)
Henrik created GROOVY-9655: -- Summary: Groovy 3.0.5 regression with the "with" keyword Key: GROOVY-9655 URL: https://issues.apache.org/jira/browse/GROOVY-9655 Project: Groovy Issue Type: Bug Affe

[jira] [Commented] (GROOVY-7701) org.codehaus.groovy.runtime.typehandling.GroovyCastException in Groovy ".with { ... }" - Block

2020-07-24 Thread Henrik (Jira)
[ https://issues.apache.org/jira/browse/GROOVY-7701?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17164232#comment-17164232 ] Henrik commented on GROOVY-7701: Since this was the only issue relating to {{with}} in

[GitHub] [groovy] blackdrag commented on pull request #1322: GROOVY-9637: Improve the performance of GString

2020-07-24 Thread GitBox
blackdrag commented on pull request #1322: URL: https://github.com/apache/groovy/pull/1322#issuecomment-663390620 This again puts a big question mark on the split between GString and GStringImpl. I think code like those instanceof checks should be avoided. Could we shortly summarize the go