[GROOVY-5829] wasted work in "AnnotationVisitor.checkIfValidEnumConstsAreUsed"

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.0.5
• Fix Version/s: 2.0.6
• Flags: Patch
• Assignee: Jochen Theodorou
• Reporter: Adrian Nistor
• Created: 29/Nov/12 10:41
• Updated: 05/Apr/15 14:43
• Resolved: 30/Nov/12 12:38

Description

The problem appears in Groovy 2.0.5 and in revision 740ae7c.. I
attached a one-line patch that fixes it.

In method "AnnotationVisitor.checkIfValidEnumConstsAreUsed", the loop
over "attributes.entrySet()" should break immediately after "ok" is
set to "false". All the iterations after "ok" is set to "false" do
not perform any useful work, at best they just set "ok" again to
"false".