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 2019/07/22 04:22:02 UTC

[jira] [Closed] (GROOVY-9054) @groovy.transform.builder.Builder should generate @Generated annotations for build(), builder() and setter methods

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

Paul King closed GROOVY-9054.
-----------------------------

> @groovy.transform.builder.Builder should generate @Generated annotations for build(), builder() and setter methods
> ------------------------------------------------------------------------------------------------------------------
>
>                 Key: GROOVY-9054
>                 URL: https://issues.apache.org/jira/browse/GROOVY-9054
>             Project: Groovy
>          Issue Type: Bug
>          Components: Compiler
>    Affects Versions: 2.5.7
>            Reporter: Jan Wloka
>            Assignee: Paul King
>            Priority: Major
>             Fix For: 2.5.8, 3.0.0-beta-2
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Using the @groovy.transform.builder.Builder with SimpleStrategy yields the following problems.
> Groovy input
> {code:java}
> @groovy.transform.builder.Builder
> class GroovyBuilderClassTarget {
>     @groovy.transform.builder.Builder(builderStrategy= SimpleStrategy)
>     class Person {
>         String first
>         String last
>         Integer born
>     }
>     static void main(String[] args) {
>         new GroovyBuilderClassTarget()
>     }
> }
> {code}
> Groovy output:
> {code:java}
> @Builder
> public class GroovyBuilderClassTarget implements GroovyObject {
>     @Generated
>     public GroovyBuilderClassTarget() {
>         CallSite[] var1 = $getCallSiteArray();
>         super();
>         MetaClass var2 = this.$getStaticMetaClass();
>         this.metaClass = var2;
>     }
>     public static void main(String... args) {
>         CallSite[] var1 = $getCallSiteArray();
>         var1[0].callConstructor(GroovyBuilderClassTarget.class);
>     }
>     @Generated
>     public static GroovyBuilderClassTarget.GroovyBuilderClassTargetBuilder builder() {
>         CallSite[] var0 = $getCallSiteArray();
>         return (GroovyBuilderClassTarget.GroovyBuilderClassTargetBuilder)ScriptBytecodeAdapter.castToType(var0[1].callConstructor(GroovyBuilderClassTarget.GroovyBuilderClassTargetBuilder.class), GroovyBuilderClassTarget.GroovyBuilderClassTargetBuilder.class);
>     }
>     @Builder(
>         builderStrategy = SimpleStrategy.class
>     )
>     public class Person implements GroovyObject {
>         private String first;
>         private String last;
>         private Integer born;
>         @Generated
>         public Person() {
>             CallSite[] var2 = $getCallSiteArray();
>             super();
>             MetaClass var4 = this.$getStaticMetaClass();
>             this.metaClass = var4;
>         }
>         @Generated
>         public GroovyBuilderClassTarget.Person setFirst(String first) {
>             CallSite[] var2 = $getCallSiteArray();
>             this.first = first;
>             return this;
>         }
>         @Generated
>         public GroovyBuilderClassTarget.Person setLast(String last) {
>             CallSite[] var2 = $getCallSiteArray();
>             this.last = last;
>             return this;
>         }
>         @Generated
>         public GroovyBuilderClassTarget.Person setBorn(Integer born) {
>             CallSite[] var2 = $getCallSiteArray();
>             this.born = born;
>             return this;
>         }
>         @Generated
>         public String getFirst() {
>             return this.first;
>         }
>         @Generated
>         public String getLast() {
>             return this.last;
>         }
>         @Generated
>         public Integer getBorn() {
>             return this.born;
>         }
>     }
>     @Generated
>     public static class GroovyBuilderClassTargetBuilder implements GroovyObject {
>         @Generated
>         public GroovyBuilderClassTargetBuilder() {
>             CallSite[] var1 = $getCallSiteArray();
>             super();
>             MetaClass var2 = this.$getStaticMetaClass();
>             this.metaClass = var2;
>         }
>         @Generated
>         public GroovyBuilderClassTarget build() {
>             CallSite[] var1 = $getCallSiteArray();
>             GroovyBuilderClassTarget _theGroovyBuilderClassTarget = (GroovyBuilderClassTarget)ScriptBytecodeAdapter.castToType(var1[0].callConstructor(GroovyBuilderClassTarget.class), GroovyBuilderClassTarget.class);
>             return _theGroovyBuilderClassTarget;
>         }
>     }
> }
> {code}
> I'd expect the following methods to have an @Generated annotation:
> public static GroovyBuilderClassTarget.GroovyBuilderClassTargetBuilder builder()
> public GroovyBuilderClassTarget.Person setFirst(String first)
> public GroovyBuilderClassTarget.Person setLast(String last)
> public GroovyBuilderClassTarget.Person setBorn(Integer born)
> public GroovyBuilderClassTarget build()
> Found during Hackergarten Zurich.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)