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

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

Revision: 1463
          http://fb-contrib.svn.sourceforge.net/fb-contrib/?rev=1463&view=rev
Author:   dbrosius
Date:     2010-01-10 17:33:17 +0000 (Sun, 10 Jan 2010)

Log Message:
-----------
don't report in a lookup or table switch

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

Modified: trunk/fb-contrib/src/com/mebigfatguy/fbcontrib/detect/PoorlyDefinedParameter.java
===================================================================
--- trunk/fb-contrib/src/com/mebigfatguy/fbcontrib/detect/PoorlyDefinedParameter.java	2010-01-10 17:16:34 UTC (rev 1462)
+++ trunk/fb-contrib/src/com/mebigfatguy/fbcontrib/detect/PoorlyDefinedParameter.java	2010-01-10 17:33:17 UTC (rev 1463)
@@ -169,19 +169,20 @@
 	                state = State.SAW_NOTHING;
 	            break;
 	        }
-	        if ((seen >= IFEQ) && (seen <= IF_ACMPNE)) {
-	        	downwardBranchTarget = getBranchTarget();
-	        	if (downwardBranchTarget < getPC())
-	        		downwardBranchTarget = -1;
+	        if (downwardBranchTarget == -1) {
+		        if ((seen >= IFEQ) && (seen <= IF_ACMPNE)) {
+		        	downwardBranchTarget = getBranchTarget();
+		        	if (downwardBranchTarget < getPC())
+		        		downwardBranchTarget = -1;
+		        } else if ((seen == LOOKUPSWITCH) || (seen == TABLESWITCH)) {
+		        	downwardBranchTarget = this.getDefaultSwitchOffset() + getPC();
+		        } 
 	        }
 	    } else {
     		state = State.SAW_NOTHING;
     		if (getPC() >= downwardBranchTarget)
     			downwardBranchTarget = -1;
     	}
-        
-        
-
     }
     
     private static class BugInfo {


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