[OWB-637] [perf] ELContextStore.destroyDependents() creates unnecessary HashMap$KeyIterator instances

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 1.1.4
• Environment: OWB trunk, 1.1.4-SNAPSHOT
• Assignee: Martin Kočí
• Reporter: Martin Kočí
• Created: 22/Dec/11 20:49
• Updated: 14/Apr/12 07:35
• Resolved: 24/Jan/12 17:54

Description

org.apache.webbeans.el.ELContextStore.destroyDependents() creates many instances of java.util.HashMap$KeyIterator - it uses directly:

Set<Bean<?>> beans = dependentObjects.keySet();
for(Bean<?> bean : beans)

simple if-check optimize it for situation, when user does not use named dependent beans:
if (dependentObjects.size() > 0) {
Set<Bean<?>> beans = dependentObjects.keySet();
for(Bean<?> bean : beans)