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

Jan Wloka updated GROOVY-9054:
------------------------------
    Description: 
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.

  was:
Using the @groovy.transform.builder.Builder with SimpleStrategy yields the 
following problems.

Groovy input
{code}
@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}
@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.


> @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.6
>            Reporter: Jan Wloka
>            Priority: Major
>
> 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.3#76005)

Reply via email to