[Fb-contrib-commit] SF.net SVN: fb-contrib:[1102] trunk/fb-contrib/src/com/mebigfatguy/ fbcontrib/detect/BloatedAssignmentScope.java

[<dbr...@us...>]

Revision: 1102
          http://fb-contrib.svn.sourceforge.net/fb-contrib/?rev=1102&view=rev
Author:   dbrosius
Date:     2009-02-22 07:19:47 +0000 (Sun, 22 Feb 2009)

Log Message:
-----------
fix stutter

Modified Paths:
--------------
    trunk/fb-contrib/src/com/mebigfatguy/fbcontrib/detect/BloatedAssignmentScope.java

Modified: trunk/fb-contrib/src/com/mebigfatguy/fbcontrib/detect/BloatedAssignmentScope.java
===================================================================
--- trunk/fb-contrib/src/com/mebigfatguy/fbcontrib/detect/BloatedAssignmentScope.java	2009-02-22 03:48:02 UTC (rev 1101)
+++ trunk/fb-contrib/src/com/mebigfatguy/fbcontrib/detect/BloatedAssignmentScope.java	2009-02-22 07:19:47 UTC (rev 1102)
@@ -258,8 +258,8 @@
 			int pc = getPC();
 			int[] offsets = getSwitchOffsets();
 			List<Integer> targets = new ArrayList<Integer>();
-			for (int i = 0; i < offsets.length; i++)
-				targets.add(Integer14.valueOf(offsets[i] + pc));
+			for (int offset : offsets)
+				targets.add(Integer14.valueOf(offset + pc));
 			Integer defOffset = Integer14.valueOf(getDefaultSwitchOffset() + pc);
 			if (!targets.contains(defOffset))
 				targets.add(defOffset);
@@ -273,7 +273,7 @@
 				lastTarget = nextTarget;
 			}
 			switchTargets.addAll(targets);
-		} else if ((seen == INVOKEVIRTUAL) || (seen == INVOKEVIRTUAL)) {
+		} else if ((seen == INVOKEVIRTUAL) || (seen == INVOKEINTERFACE)) {
 		    if ("wasNull".equals(getNameConstantOperand())
             &&  "()Z".equals(getSigConstantOperand()))
                 dontReport = true;


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.