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

Sebastien Arod commented on GROOVY-4654:
----------------------------------------

Ok Thanks for the clarification. 
By the way I totally understand the concern around ThreadLocal that's why I 
made the handle of cycle optional.

I'll recreate the pull request ASAP.

> groovy.transform.ToString annotation does not handle cycles
> -----------------------------------------------------------
>
>                 Key: GROOVY-4654
>                 URL: https://issues.apache.org/jira/browse/GROOVY-4654
>             Project: Groovy
>          Issue Type: Bug
>          Components: groovy-jdk
>    Affects Versions: 1.8-beta-3
>            Reporter: Travis Schneeberger
>              Labels: contrib
>         Attachments: Bar.groovy, Foo.groovy, ToStringCycleTest.groovy
>
>
> The groovy.transform.ToString AST transformation does not handle object 
> graphs with cycles.  See the attached junit tests.  This is a pretty serious 
> issue because these object structures are common with some frameworks (like 
> orm tools) 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to