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)