[prev in list] [next in list] [prev in thread] [next in thread] 

List:       groovy-scm
Subject:    [groovy-scm] [jira] Commented: (GROOVY-3713) @Immutable should not override
From:       "Roshan Dawrani (JIRA)" <jira () codehaus ! org>
Date:       2009-08-31 13:26:41
Message-ID: 16001015.92541251725201687.JavaMail.haus-jira () codehaus01 ! managed ! contegix ! com
[Download RAW message or body]


    [ http://jira.codehaus.org/browse/GROOVY-3713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=189231#action_189231 \
] 

Roshan Dawrani commented on GROOVY-3713:
----------------------------------------

I will try the toString() change in the meanwhile and wait for more comments on \
whether users can be told up-front that they should not provide hashCode() / equals() \
with @Immutable. Some change there is needed anyway because if they provide, class \
generated will run into ClassFormatError due to repetitive hashCode() / equals() \
methods.

> @Immutable should not override toString() if provided in source
> ---------------------------------------------------------------
> 
> Key: GROOVY-3713
> URL: http://jira.codehaus.org/browse/GROOVY-3713
> Project: groovy
> Issue Type: Bug
> Affects Versions: 1.7-beta-1
> Reporter: Dierk Koenig
> Fix For: 1.7-beta-2
> 
> 
> {code}
> @Immutable class MyClass {
> String toString() { 'some return value' }
> }
> new MyClass()
> {code}
> throws
> {code}
> Repetitive method name/signature for method 'java.lang.String toString()' in class \
> 'MyClass'. at line: 2, column: 5
> Repetitive method name/signature for method 'java.lang.String toString()' in class \
> 'MyClass'. at line: -1, column: -1
> {code}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: \
                http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

<hr/>
<p>
To unsubscribe from this list please visit:
</p>
<p>
    <a href="http://xircles.codehaus.org/manage_email">http://xircles.codehaus.org/manage_email</a>



[prev in list] [next in list] [prev in thread] [next in thread]