You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@groovy.apache.org by "Paul King (JIRA)" <ji...@apache.org> on 2018/10/08 10:29:00 UTC

[jira] [Commented] (GROOVY-8833) @Canonical creates invalid hashCode

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

Paul King commented on GROOVY-8833:
-----------------------------------

Well spotted. This is now fixed.

> @Canonical creates invalid hashCode
> -----------------------------------
>
>                 Key: GROOVY-8833
>                 URL: https://issues.apache.org/jira/browse/GROOVY-8833
>             Project: Groovy
>          Issue Type: Bug
>          Components: Compiler
>    Affects Versions: 2.5.2
>            Reporter: Kenzie Togami
>            Priority: Critical
>             Fix For: 3.0.0-alpha-4, 2.5.3
>
>
> Applying {{@Canonical}} to a class will generate an invalid hashCode method that throws a runtime cast exception. This does not happen if you apply the individual annotations by themselves.
> These tests can be injected into a Groovy test class to see the problem: [https://gist.github.com/kenzierocks/fb0932756c3955a7ddbc0b710791af11]
> I suspect the underlying cause is that {{@EqualsAndHashCode}} does not set {{accessedVariable}} on its {{_result}} variable, which causes the verifier to overwrite it with {{toString}}'s {{_result}} variable (see {{FinalVariableAnalyzer#fixVar}}). This results in an invalid cast. I observed this behavior while debugging the compiler. I'm not sure why this doesn't also happen if the two annotations are applied by themselves, rather than via {{@Canonical}}.
>  
> I was able to fix this problem by setting {{_result}} to reference itself in the AST, but perhaps that's not the right fix here.



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