[jira] [Resolved] (GROOVY-9176) Syntactic sugar for set method calls in conflict with class member declarations

["Paul King (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/GROOVY-9176?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Paul King resolved GROOVY-9176.
-------------------------------
       Resolution: Fixed
         Assignee: Paul King
    Fix Version/s: 3.0.0-beta-2
                   2.5.8

Thanks Eric, I have merged the PR. For those reading this later, the compiler still rejects some classes that may have accidentally worked in earlier versions of Groovy but there is now a better error message indicating the reason behind the stricter checking.

> Syntactic sugar for set method calls in conflict with class member declarations
> -------------------------------------------------------------------------------
>
>                 Key: GROOVY-9176
>                 URL: https://issues.apache.org/jira/browse/GROOVY-9176
>             Project: Groovy
>          Issue Type: Bug
>          Components: Compiler
>    Affects Versions: 3.0.0-beta-1
>            Reporter: David Ekholm
>            Assignee: Paul King
>            Priority: Minor
>             Fix For: 2.5.8, 3.0.0-beta-2
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> Groovy 3.0.0 beta 1 has broken code that earlier worked in Alpha 4:
> Say you have a Java class with a setTitle(String) method. Now you override this class with a Groovy class. In it you declare the following member variable:
> JLabel title = new JLabel("skin");
> This breaks the Groovy compiler in v3.0.0 beta 1 (but not alpha 4) with the following error:
> The return type of javax.swing.JLabel getTitle() in Script34$1$12 is incompatible with java.lang.String in se.datadosen.component.ControlPanel
> . At [-1:-1]  @ line -1, column -1.
> Two things are noteworthy here:
> 1) The line number references (-1) are broken
> 2) Instead of letting the declared "title" variable hide the ability to call setTitle on the base class using the syntactic sugar of Groovy (foo = being syntactic sugar for setFoo(...)), it now bails out with this error, breaking Java compatibility. I expected this declaration to be allowed and simply hide the ability to call setTitle on the base class via that syntactic sugar.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)