[jira] [Comment Edited] (GROOVY-9342) SC: Lambda in static initializer that uses static field on LHS of += produces errors

["Eric Milles (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/GROOVY-9342?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17000318#comment-17000318 ] 

Eric Milles edited comment on GROOVY-9342 at 12/19/19 7:25 PM:
---------------------------------------------------------------

"Cannot resolve method int#plus(Object)..." is still an issue because  {{StaticTypeCheckingVisitor#inferSAMType}} cannot collect generics correctly from {{forEach(Consumer<? super T> c)}} and so the param type is Object.

PR #1126 fixes the assignment case, which is handled by separate methods:
{code:groovy}
int sum = 0
java.util.function.Consumer<? super Integer> add = i -> sum += i
{code}



was (Author: emilles):
"Cannot resolve method int#plus(Object)..." is still an issue because  {{StaticTypeCheckingVisitor#inferSAMType}} cannot collect generics correctly from {{forEach(Consumer<? extends T> c)}}.

PR #1126 fixes the assignment case, which is handled by separate methods:
{code:groovy}
int sum = 0
java.util.function.Consumer<? super Integer> add = i -> sum += i
{code}


> SC: Lambda in static initializer that uses static field on LHS of += produces errors
> ------------------------------------------------------------------------------------
>
>                 Key: GROOVY-9342
>                 URL: https://issues.apache.org/jira/browse/GROOVY-9342
>             Project: Groovy
>          Issue Type: Bug
>    Affects Versions: 3.0.0-rc-2
>            Reporter: Eric Milles
>            Assignee: Eric Milles
>            Priority: Major
>             Fix For: 3.0.0-rc-3
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Follow up to GROOVY-9332 and GROOVY-9333.  Consider the following:
> {code:groovy}
> @groovy.transform.CompileStatic
> class Test1 {
>     static int acc = 1
>     static { [1, 2, 3].forEach(e -> acc += e) }
> }
> assert Test1.acc == 7
> {code}
> {code}
> org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
>     TestScript60.groovy: 5: [Static type checking] - Cannot find matching method int#plus(java.lang.Object). Please check if the declared type is correct and if the method exists.
>      @ line 5, column 49.
>        atic { [1, 2, 3].forEach(e -> acc += e)
>                                      ^
>     1 error
>         at org.codehaus.groovy.control.ErrorCollector.failIfErrors(ErrorCollector.java:287)
>         at org.codehaus.groovy.control.CompilationUnit.applyToPrimaryClassNodes(CompilationUnit.java:1091)
>         at org.codehaus.groovy.control.CompilationUnit.doPhaseOperation(CompilationUnit.java:633)
>         at org.codehaus.groovy.control.CompilationUnit.processPhaseOperations(CompilationUnit.java:612)
>         at org.codehaus.groovy.control.CompilationUnit.compile(CompilationUnit.java:589)
>         at groovy.lang.GroovyClassLoader.doParseClass(GroovyClassLoader.java:367)
>         at groovy.lang.GroovyClassLoader.lambda$parseClass$2(GroovyClassLoader.java:310)
>         ...
> {code}
> and a different error if lambda uses specific parameter type:
> {code:groovy}
> @groovy.transform.CompileStatic
> class Test1 {
>     static int acc = 1
>     static { [1, 2, 3].forEach((Integer i) -> acc += i) }
> }
> assert Test1.acc == 7
> {code}
> {code}
> java.lang.ExceptionInInitializerError\r\n
> 	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)\r\n
> 	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)\r\n
> 	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)\r\n
> 	at java.lang.reflect.Method.invoke(Method.java:498)\r\n
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)