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

Paul King edited comment on GROOVY-5936 at 8/4/16 5:01 AM:
-----------------------------------------------------------

The {{dropRight}} method now has an iterator-backed implementation.


was (Author: paulk):
dropRight has been added

> java.lang.OutOfMemoryError through Iterator.collectEntries and others (part 
> 2: handle others)
> ---------------------------------------------------------------------------------------------
>
>                 Key: GROOVY-5936
>                 URL: https://issues.apache.org/jira/browse/GROOVY-5936
>             Project: Groovy
>          Issue Type: Bug
>          Components: groovy-jdk
>    Affects Versions: 2.0.4
>            Reporter: Alexander Veit
>
> DGM.collectEntries(Iterator, ...) naively calls toList(Iterator<T> self) to 
> convert the Iterator to a list. There's no need to do this since iterator 
> suffices to collect the entries. Even worse, toList uses ArrayList what may 
> lead to multiple reallocations when the ArrayList's capacity must be enlarged.
> Error:
> Caught: java.lang.OutOfMemoryError: GC overhead limit exceeded
> The same problems may occur when calling other methods that internally 
> convert Iterator to lists.



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

Reply via email to