[GROOVY-4800] Avoid too many temporary arrays with each and eachWithIndex

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.9-beta-1
• Component/s: groovy-jdk
• Assignee: Cédric Champeau
• Reporter: Cédric Champeau
• Created: 28/Apr/11 04:43
• Updated: 21/Jul/11 19:06
• Resolved: 28/Apr/11 05:05

Description

In DGM, each and eachWithIndex methods delegate to closure.call(argument) which creates a temporary array for each iteration. We can create the temporary array in the each method itself, then call closure.call(array) instead.

Another optimization would be to cache the MetaMethod used by the closure, but it would lead to slightly different semantics so we'll leave this for later.