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

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

Revision: 588
Author:   dbrosius
Date:     2006-08-05 15:16:24 -0700 (Sat, 05 Aug 2006)
ViewCVS:  http://svn.sourceforge.net/fb-contrib/?rev=588&view=rev

Log Message:
-----------
fix LII due to changes in OpcodeStack, not setting a variable constant to 0 when the assignment is a for loop. Use couldBeZero() instead.

Modified Paths:
--------------
    trunk/fb-contrib/src/com/mebigfatguy/fbcontrib/detect/ListIndexedIterating.java
Modified: trunk/fb-contrib/src/com/mebigfatguy/fbcontrib/detect/ListIndexedIterating.java
===================================================================
--- trunk/fb-contrib/src/com/mebigfatguy/fbcontrib/detect/ListIndexedIterating.java	2006-08-05 22:05:43 UTC (rev 587)
+++ trunk/fb-contrib/src/com/mebigfatguy/fbcontrib/detect/ListIndexedIterating.java	2006-08-05 22:16:24 UTC (rev 588)
@@ -248,8 +248,7 @@
 								fl.setLoopState(LOOP_IN_BODY_WITH_GET);
 								if (stack.getStackDepth() > 1) {
 									OpcodeStack.Item itm = stack.getStackItem(0);
-									Integer initValue = (Integer)itm.getConstant();
-									if ((initValue == null) || (initValue.intValue() != 0))
+									if (!itm.couldBeZero())
 										it.remove();
 									else {
 										itm = stack.getStackItem(1);


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