[jira] [Commented] (GROOVY-8101) DefaultGroovyMethods.plus() copies internal state of the left Map

2017-03-01 Thread Jochen Theodorou (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15891687#comment-15891687 ] Jochen Theodorou commented on GROOVY-8101: -- [~nickknw] What I wanted to say is the following: if

[jira] [Commented] (GROOVY-8101) DefaultGroovyMethods.plus() copies internal state of the left Map

2017-03-01 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15891563#comment-15891563 ] Paul King commented on GROOVY-8101: --- Sounds like you want: {code} Map combinedMaps1 =

[jira] [Commented] (GROOVY-8101) DefaultGroovyMethods.plus() copies internal state of the left Map

2017-03-01 Thread Nick Knowlson (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15891342#comment-15891342 ] Nick Knowlson commented on GROOVY-8101: --- > I think what Groovy is doing is proper, trying to return

[jira] [Resolved] (GROOVY-8102) Does/could Groovy support multiple variable definitions in for loop?

2017-03-01 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8102?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Sun resolved GROOVY-8102. Resolution: Fixed Fix Version/s: 3.0 As John found, the improvement has been completed in

[jira] [Assigned] (GROOVY-8102) Does/could Groovy support multiple variable definitions in for loop?

2017-03-01 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8102?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Sun reassigned GROOVY-8102: -- Assignee: Daniel Sun > Does/could Groovy support multiple variable definitions in for loop? >

[jira] [Commented] (GROOVY-8101) DefaultGroovyMethods.plus() copies internal state of the left Map

2017-03-01 Thread John Wagenleitner (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15891243#comment-15891243 ] John Wagenleitner commented on GROOVY-8101: --- {quote}It doesn't provide a custom clone method or

[jira] [Commented] (GROOVY-8101) DefaultGroovyMethods.plus() copies internal state of the left Map

2017-03-01 Thread Nick Knowlson (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15891181#comment-15891181 ] Nick Knowlson commented on GROOVY-8101: --- Okay, I can see how it would be expected behaviour if you

[GitHub] groovy pull request #506: Fixed documentation of default Grape root director...

2017-03-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/groovy/pull/506 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[jira] [Commented] (GROOVY-8101) DefaultGroovyMethods.plus() copies internal state of the left Map

2017-03-01 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15891038#comment-15891038 ] Paul King commented on GROOVY-8101: --- I think what Jochen is saying is that in the example you have

[jira] [Commented] (GROOVY-8101) DefaultGroovyMethods.plus() copies internal state of the left Map

2017-03-01 Thread Nick Knowlson (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15890967#comment-15890967 ] Nick Knowlson commented on GROOVY-8101: --- > that means your map has a clone method and using that

[jira] [Commented] (GROOVY-8102) Does/could Groovy support multiple variable definitions in for loop?

2017-03-01 Thread John Wagenleitner (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15890775#comment-15890775 ] John Wagenleitner commented on GROOVY-8102: --- I believe this is supported in the anltr4 "parrot"

[GitHub] groovy pull request #506: Fixed documentation of default Grape root director...

2017-03-01 Thread keeganwitt
GitHub user keeganwitt opened a pull request: https://github.com/apache/groovy/pull/506 Fixed documentation of default Grape root directory You can merge this pull request into a Git repository by running: $ git pull https://github.com/keeganwitt/groovy fix-grape-doc

[jira] [Created] (GROOVY-8103) STC: matching return type of one static method to parameter type of another

2017-03-01 Thread Eric Milles (JIRA)
Eric Milles created GROOVY-8103: --- Summary: STC: matching return type of one static method to parameter type of another Key: GROOVY-8103 URL: https://issues.apache.org/jira/browse/GROOVY-8103 Project:

[jira] [Created] (GROOVY-8102) Does/could Groovy support multiple variable definitions in for loop?

2017-03-01 Thread Eric Milles (JIRA)
Eric Milles created GROOVY-8102: --- Summary: Does/could Groovy support multiple variable definitions in for loop? Key: GROOVY-8102 URL: https://issues.apache.org/jira/browse/GROOVY-8102 Project: Groovy

[jira] [Resolved] (GROOVY-7402) Compiler NPE Related To AST Transformations And Traits

2017-03-01 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7402?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Paul King resolved GROOVY-7402. --- Resolution: Fixed Fix Version/s: 2.4.10 Proposed PR merged. Instead of a ClassFormatError or

[jira] [Commented] (GROOVY-7402) Compiler NPE Related To AST Transformations And Traits

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

[GitHub] groovy pull request #499: GROOVY-7402: Compiler NPE Related To AST Transform...

2017-03-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/groovy/pull/499 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is