[jira] [Resolved] (GROOVY-6631) Static compilation fails around byte arrays

2018-05-16 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6631?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Sun resolved GROOVY-6631. Resolution: Not A Problem Assignee: Daniel Sun `i / 2` will yield a `BigDecimal`, e.g. 1 / 2

[jira] [Resolved] (GROOVY-6632) CompileStatic selects wrong constructor when GString is a parameter

2018-05-16 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6632?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Sun resolved GROOVY-6632. Resolution: Fixed Assignee: Daniel Sun Fix Version/s: 2.5.0-rc-3

[GitHub] groovy pull request #708: GROOVY-6668: Static compiler doesn't coerce GStrin...

2018-05-16 Thread danielsun1106
Github user danielsun1106 commented on a diff in the pull request: https://github.com/apache/groovy/pull/708#discussion_r188837462 --- Diff: src/test/groovy/bugs/Groovy6668Bug.groovy --- @@ -0,0 +1,65 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

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

[GitHub] groovy pull request #708: GROOVY-6668: Static compiler doesn't coerce GStrin...

2018-05-16 Thread paulk-asert
Github user paulk-asert commented on a diff in the pull request: https://github.com/apache/groovy/pull/708#discussion_r188832656 --- Diff: src/test/groovy/bugs/Groovy6668Bug.groovy --- @@ -0,0 +1,65 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

[GitHub] groovy pull request #708: GROOVY-6668: Static compiler doesn't coerce GStrin...

2018-05-16 Thread danielsun1106
Github user danielsun1106 commented on a diff in the pull request: https://github.com/apache/groovy/pull/708#discussion_r188826337 --- Diff: src/test/groovy/bugs/Groovy6668Bug.groovy --- @@ -0,0 +1,65 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478416#comment-16478416 ] ASF GitHub Bot commented on GROOVY-6668: Github user danielsun1106 commented on a diff in the

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478412#comment-16478412 ] ASF GitHub Bot commented on GROOVY-6668: Github user danielsun1106 commented on a diff in the

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478388#comment-16478388 ] ASF GitHub Bot commented on GROOVY-6668: Github user blackdrag commented on a diff in the pull

[GitHub] groovy pull request #708: GROOVY-6668: Static compiler doesn't coerce GStrin...

2018-05-16 Thread blackdrag
Github user blackdrag commented on a diff in the pull request: https://github.com/apache/groovy/pull/708#discussion_r188822722 --- Diff: src/test/groovy/bugs/Groovy6668Bug.groovy --- @@ -0,0 +1,65 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478382#comment-16478382 ] ASF GitHub Bot commented on GROOVY-6668: Github user blackdrag commented on a diff in the pull

[GitHub] groovy pull request #708: GROOVY-6668: Static compiler doesn't coerce GStrin...

2018-05-16 Thread blackdrag
Github user blackdrag commented on a diff in the pull request: https://github.com/apache/groovy/pull/708#discussion_r188821809 --- Diff: src/main/java/org/codehaus/groovy/transform/stc/StaticTypeCheckingVisitor.java --- @@ -4101,13 +4101,17 @@ protected static ClassNode

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478381#comment-16478381 ] Daniel Sun commented on GROOVY-6668: [~melix] Jochen has provided some examples to statement the

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478380#comment-16478380 ] ASF GitHub Bot commented on GROOVY-6668: Github user blackdrag commented on a diff in the pull

[GitHub] groovy pull request #708: GROOVY-6668: Static compiler doesn't coerce GStrin...

2018-05-16 Thread blackdrag
Github user blackdrag commented on a diff in the pull request: https://github.com/apache/groovy/pull/708#discussion_r188821671 --- Diff: src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticInvocationWriter.java --- @@ -468,7 +469,11 @@ protected void loadArguments(List

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478371#comment-16478371 ] ASF GitHub Bot commented on GROOVY-6668: GitHub user danielsun1106 opened a pull request:

[GitHub] groovy pull request #708: GROOVY-6668: Static compiler doesn't coerce GStrin...

2018-05-16 Thread danielsun1106
GitHub user danielsun1106 opened a pull request: https://github.com/apache/groovy/pull/708 GROOVY-6668: Static compiler doesn't coerce GString for getAt() call You can merge this pull request into a Git repository by running: $ git pull https://github.com/danielsun1106/groovy

[jira] [Commented] (GROOVY-8570) Support Warnings in Groovy Compiler

2018-05-16 Thread Jochen Theodorou (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478349#comment-16478349 ] Jochen Theodorou commented on GROOVY-8570: -- "Groovy-to-idiomatic-Groovy converter" is what I did

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread Jochen Theodorou (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478329#comment-16478329 ] Jochen Theodorou commented on GROOVY-6668: -- To add on the written before... so while I

[jira] [Comment Edited] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread Jochen Theodorou (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478299#comment-16478299 ] Jochen Theodorou edited comment on GROOVY-6668 at 5/17/18 12:50 AM:

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread Jochen Theodorou (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478299#comment-16478299 ] Jochen Theodorou commented on GROOVY-6668: -- [~melix] to get this right... {code} def foo(String

[jira] [Commented] (GROOVY-8570) Support Warnings in Groovy Compiler

2018-05-16 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478283#comment-16478283 ] Paul King commented on GROOVY-8570: --- But there has been discussion about having a special semantics

[jira] [Commented] (GROOVY-8570) Support Warnings in Groovy Compiler

2018-05-16 Thread mgroovy (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16478097#comment-16478097 ] mgroovy commented on GROOVY-8570: - @[~paulk]: But how is someone to tell that "var" is bad style, and

[jira] [Commented] (GROOVY-8509) SC: error for call to protected method from same package

2018-05-16 Thread John Wagenleitner (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8509?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16477881#comment-16477881 ] John Wagenleitner commented on GROOVY-8509: --- Thanks for the suggested patch. {quote}Why does

[jira] [Commented] (GROOVY-8562) Wrong Closure delegation for property access when using @CompileStatic and DELEGATE_FIRST / DELEGATE_ONLY

2018-05-16 Thread Michael Arndt (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16477880#comment-16477880 ] Michael Arndt commented on GROOVY-8562: --- Looked into it. I fixed the first case, but it breaks 28

[jira] [Commented] (GROOVY-8562) Wrong Closure delegation for property access when using @CompileStatic and DELEGATE_FIRST / DELEGATE_ONLY

2018-05-16 Thread Michael Arndt (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16477872#comment-16477872 ] Michael Arndt commented on GROOVY-8562: --- Good to know! Working without Spock for data-driven makes

[jira] [Commented] (GROOVY-8509) SC: error for call to protected method from same package

2018-05-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8509?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16477858#comment-16477858 ] ASF GitHub Bot commented on GROOVY-8509: GitHub user jwagenleitner opened a pull request:

[GitHub] groovy pull request #707: GROOVY-8509: SC error call to protected method fro...

2018-05-16 Thread jwagenleitner
GitHub user jwagenleitner opened a pull request: https://github.com/apache/groovy/pull/707 GROOVY-8509: SC error call to protected method from same package Because they were closely related, this PR also includes the following: 1. A change to the fix introduced by

[jira] [Commented] (GROOVY-8587) Calling super in an overridden method results in MissingMethodException when method indirectly inherited from trait

2018-05-16 Thread Jonathan Cottrill (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16477586#comment-16477586 ] Jonathan Cottrill commented on GROOVY-8587: --- An ugly workaround is to replace the

[jira] [Updated] (GROOVY-8587) Calling super in an overridden method results in MissingMethodException when method indirectly inherited from trait

2018-05-16 Thread Jonathan Cottrill (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8587?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jonathan Cottrill updated GROOVY-8587: -- Description: Consider the following: {code:groovy} trait A { void doSomething() {

[jira] [Updated] (GROOVY-8587) Calling super in an overridden method results in MissingMethodException when method indirectly inherited from trait

2018-05-16 Thread Jonathan Cottrill (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8587?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jonathan Cottrill updated GROOVY-8587: -- Description: Consider the following: {code:groovy} trait A { void doSomething() {

[jira] [Updated] (GROOVY-8587) Calling super in an overridden method results in MissingMethodException when method indirectly inherited from trait

2018-05-16 Thread Jonathan Cottrill (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8587?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jonathan Cottrill updated GROOVY-8587: -- Description: Consider the following: {{trait A {}} {{    void doSomething() {}} {{   

[jira] [Updated] (GROOVY-8587) Calling super in an overridden method results in MissingMethodException when method indirectly inherited from trait

2018-05-16 Thread Jonathan Cottrill (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8587?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jonathan Cottrill updated GROOVY-8587: -- Description: Consider the following: trait A { void doSomething() {

[jira] [Updated] (GROOVY-8587) Calling super in an overridden method results in MissingMethodException when method indirectly inherited from trait

2018-05-16 Thread Jonathan Cottrill (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8587?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jonathan Cottrill updated GROOVY-8587: -- Description: Consider the following: {{trait A {}} {{    void doSomething() {}} {{   

[jira] [Created] (GROOVY-8587) Calling super in an overridden method results in MissingMethodException when method indirectly inherited from trait

2018-05-16 Thread Jonathan Cottrill (JIRA)
Jonathan Cottrill created GROOVY-8587: - Summary: Calling super in an overridden method results in MissingMethodException when method indirectly inherited from trait Key: GROOVY-8587 URL:

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread JIRA
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16477440#comment-16477440 ] Cédric Champeau commented on GROOVY-6668: - I understand the temptation, but no, we shouldn't.

[jira] [Commented] (GROOVY-6668) Static compiler doesn't coerce GString for getAt() call

2018-05-16 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-6668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16477428#comment-16477428 ] Daniel Sun commented on GROOVY-6668: The following test passes. {code:java}

[jira] [Updated] (GROOVY-8355) Instanceof inference does not work on field assigning

2018-05-16 Thread Daniel Sun (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8355?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Sun updated GROOVY-8355: --- Fix Version/s: (was: 2.5.0-rc-3) (was: 3.0.0-alpha-3)

[jira] [Updated] (GROOVY-8578) Add errorWriter property to commons-cli CliBuilder

2018-05-16 Thread Remko Popma (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Remko Popma updated GROOVY-8578: Description: This ticket proposes to modify the commons-cli version of CliBuilder, so that it

[jira] [Updated] (GROOVY-8578) Add errorWriter property to commons-cli CliBuilder

2018-05-16 Thread Remko Popma (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8578?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Remko Popma updated GROOVY-8578: Description: The picocli version of CliBuilder has an {{errorWriter}} property ({{System.err}} by