[GitHub] groovy pull request #370: Add access to declared variables in the VariableSc...

2016-07-21 Thread baybatu
Github user baybatu commented on a diff in the pull request: https://github.com/apache/groovy/pull/370#discussion_r71832411 --- Diff: src/main/org/codehaus/groovy/ast/VariableScope.java --- @@ -178,4 +178,26 @@ public Object removeReferencedClassVariable(String name) { pub

[jira] [Commented] (GROOVY-7888) Type checker infers wrong type collection plus assignment (+=) for property on Java class

2016-07-21 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7888?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388997#comment-15388997 ] Paul King commented on GROOVY-7888: --- The long-hand version is correctly handled too: {c

[GitHub] groovy pull request #366: GROOVY-7877: The Range abstraction could support n...

2016-07-21 Thread jwagenleitner
Github user jwagenleitner commented on a diff in the pull request: https://github.com/apache/groovy/pull/366#discussion_r71829235 --- Diff: src/main/groovy/lang/NumberRange.java --- @@ -0,0 +1,603 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

[jira] [Commented] (GROOVY-7877) The Range abstraction could support numeric ranges where the items in the range differ by some step size different to 1

2016-07-21 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7877?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388958#comment-15388958 ] ASF GitHub Bot commented on GROOVY-7877: Github user jwagenleitner commented on a

[GitHub] groovy pull request #366: GROOVY-7877: The Range abstraction could support n...

2016-07-21 Thread jwagenleitner
Github user jwagenleitner commented on a diff in the pull request: https://github.com/apache/groovy/pull/366#discussion_r71829147 --- Diff: src/main/groovy/lang/NumberRange.java --- @@ -0,0 +1,603 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

[jira] [Commented] (GROOVY-7877) The Range abstraction could support numeric ranges where the items in the range differ by some step size different to 1

2016-07-21 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7877?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388955#comment-15388955 ] ASF GitHub Bot commented on GROOVY-7877: Github user jwagenleitner commented on a

[jira] [Commented] (GROOVY-7877) The Range abstraction could support numeric ranges where the items in the range differ by some step size different to 1

2016-07-21 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7877?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388911#comment-15388911 ] ASF GitHub Bot commented on GROOVY-7877: Github user jwagenleitner commented on a

[GitHub] groovy pull request #366: GROOVY-7877: The Range abstraction could support n...

2016-07-21 Thread jwagenleitner
Github user jwagenleitner commented on a diff in the pull request: https://github.com/apache/groovy/pull/366#discussion_r71826775 --- Diff: src/main/groovy/lang/NumberRange.java --- @@ -0,0 +1,637 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

[jira] [Commented] (GROOVY-7877) The Range abstraction could support numeric ranges where the items in the range differ by some step size different to 1

2016-07-21 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7877?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388902#comment-15388902 ] ASF GitHub Bot commented on GROOVY-7877: Github user jwagenleitner commented on a

[GitHub] groovy pull request #366: GROOVY-7877: The Range abstraction could support n...

2016-07-21 Thread jwagenleitner
Github user jwagenleitner commented on a diff in the pull request: https://github.com/apache/groovy/pull/366#discussion_r71826135 --- Diff: src/main/groovy/lang/NumberRange.java --- @@ -0,0 +1,637 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

[jira] [Commented] (GROOVY-7889) Add autoFlush option to IndentPrinter

2016-07-21 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388874#comment-15388874 ] Paul King commented on GROOVY-7889: --- You can shorten that slightly with {{newWriter}}.

[jira] [Commented] (GROOVY-7889) Add autoFlush option to IndentPrinter

2016-07-21 Thread M. Justin (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388736#comment-15388736 ] M. Justin commented on GROOVY-7889: --- Nope, since per the documentation [PrintWriter|ht

[jira] [Comment Edited] (GROOVY-7889) Add autoFlush option to IndentPrinter

2016-07-21 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388668#comment-15388668 ] Paul King edited comment on GROOVY-7889 at 7/22/16 12:20 AM: -

[jira] [Commented] (GROOVY-7889) Add autoFlush option to IndentPrinter

2016-07-21 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388668#comment-15388668 ] Paul King commented on GROOVY-7889: --- There are numerous constructors that take a {{Prin

[jira] [Resolved] (GROOVY-7876) ClassCastException when calling DefaultTypeTransformation#compareEqual

2016-07-21 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7876?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Paul King resolved GROOVY-7876. --- Resolution: Fixed Assignee: Paul King Fix Version/s: 2.4.8 Proposed PR merged > Clas

[jira] [Commented] (GROOVY-7876) ClassCastException when calling DefaultTypeTransformation#compareEqual

2016-07-21 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15388616#comment-15388616 ] ASF GitHub Bot commented on GROOVY-7876: Github user asfgit closed the pull reque

[GitHub] groovy pull request #368: GROOVY-7876: ClassCastException when calling Defau...

2016-07-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/groovy/pull/368 --- 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 enab

[GitHub] groovy pull request #370: Add access to declared variables in the VariableSc...

2016-07-21 Thread natgabb
GitHub user natgabb opened a pull request: https://github.com/apache/groovy/pull/370 Add access to declared variables in the VariableScope For easier parsing of symbols. You can merge this pull request into a Git repository by running: $ git pull https://github.com/natgabb/groo

[jira] [Updated] (GROOVY-7890) Lack of compilation error while statically compiled the closure.

2016-07-21 Thread Damian Warszawski (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Damian Warszawski updated GROOVY-7890: -- Description: Expecting compile time error that cannot reference non-static variable fro

[jira] [Created] (GROOVY-7890) Lack of compilation error while statically compiled the closure.

2016-07-21 Thread Damian Warszawski (JIRA)
Damian Warszawski created GROOVY-7890: - Summary: Lack of compilation error while statically compiled the closure. Key: GROOVY-7890 URL: https://issues.apache.org/jira/browse/GROOVY-7890 Project: G

[jira] [Created] (GROOVY-7889) Add autoFlush option to IndentPrinter

2016-07-21 Thread M. Justin (JIRA)
M. Justin created GROOVY-7889: - Summary: Add autoFlush option to IndentPrinter Key: GROOVY-7889 URL: https://issues.apache.org/jira/browse/GROOVY-7889 Project: Groovy Issue Type: Improvement