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

ASF GitHub Bot commented on GROOVY-11309:
-----------------------------------------

PaintNinja commented on PR #2054:
URL: https://github.com/apache/groovy/pull/2054#issuecomment-1933744394

   Thanks. In that case, would you like me to add a code comment to 
ScriptBytecodeAdapter#createList, referencing this change?




> Optimise bytecode for empty list expressions
> --------------------------------------------
>
>                 Key: GROOVY-11309
>                 URL: https://issues.apache.org/jira/browse/GROOVY-11309
>             Project: Groovy
>          Issue Type: Improvement
>          Components: class generator, Compiler, performance
>    Affects Versions: 2.5.23, 3.0.20, 4.0.18, 5.0.0-alpha-5
>            Reporter: Oscar N
>            Priority: Minor
>
> I have the following code:
> {code:groovy}
> class Book {
>     String title
>     String blurb = ''
>     List<String> authors = []
> }
> {code}
> When checking the compiled output, the creation of the empty list goes 
> through ScriptBytecodeAdapter:
> {code:java}
> @Generated
> public Book() {
>     String var1 = "";
>     this.blurb = var1;
>     List var2 = ScriptBytecodeAdapter.createList(new Object[0]);
>     this.authors = var2;
>     MetaClass var3 = this.$getStaticMetaClass();
>     this.metaClass = var3;
> }
> {code}
> This case can be simplified to "List var2 = new ArrayList()", which would 
> avoid some overhead and slightly improve @POJO support.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to