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

List:       groovy-scm
Subject:    [groovy-scm] [jira] Updated: (GROOVY-3713) @Immutable should not override
From:       "Roshan Dawrani (JIRA)" <jira () codehaus ! org>
Date:       2009-08-31 18:24:41
Message-ID: 12385347.1621251743081581.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:all-tabpanel \
]

Roshan Dawrani updated GROOVY-3713:
-----------------------------------

    Attachment: 17x_Patch.txt

Attaching a patch with 

1) throwing errors for overridden equals() / hashCode()

2) allowing users to override toString() by not adding the generated toString() if it \
is provided.

> @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
> 
> Attachments: 17x_Patch.txt
> 
> 
> {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] 

Configure | About | News | Add a list | Sponsored by KoreLogic