use new targeter set rather than array

1 files changed, 8 insertions, 8 deletions

diff --git a/weaver/src/org/aspectj/weaver/bcel/Range.java b/weaver/src/org/aspectj/weaver/bcel/Range.java
index c6fd1aa3c..c0d46d8f0 100644
--- a/weaver/src/org/aspectj/weaver/bcel/Range.java
+++ b/weaver/src/org/aspectj/weaver/bcel/Range.java
@@ -13,6 +13,7 @@
 
 package org.aspectj.weaver.bcel;
 
+import java.util.Iterator;
 import java.util.Map;
 
 import org.aspectj.apache.bcel.generic.Instruction;
@@ -196,17 +197,16 @@ abstract class Range implements InstructionTargeter {
     protected static final Range getRange(InstructionHandle ih) {
         // assert isRangeHandle(ih)
 		Range ret = null;
-        InstructionTargeter[] targeters = ih.getTargeters();
-        if (targeters!=null) {
-        for (int i = targeters.length - 1; i >= 0; i--) {
-            if (targeters[i] instanceof Range) {
-            	Range r = (Range) targeters[i];
+		Iterator tIter = ih.getTargeters().iterator();
+		while (tIter.hasNext()) {
+			InstructionTargeter targeter = (InstructionTargeter)tIter.next();
+            if (targeter instanceof Range) {
+            	Range r = (Range) targeter;
 				if (r.getStart() != ih && r.getEnd() != ih) continue;
-				if (ret != null) throw new BCException("multiple ranges on same range handle: " + ret + ",  " + targeters[i]);
-            	ret = (Range) targeters[i];
+				if (ret != null) throw new BCException("multiple ranges on same range handle: " + ret + ",  " + targeter);
+            	ret = r;
             }
         }
-        }
 		if (ret == null) {
 			throw new BCException("shouldn't happen");
 		}