InvokeOperator (method interpreter): Fixed call to ClassInfo.forName.

TransformExceptionHandlers:  Fixed a bug regarding call to removeBadJSR.


git-svn-id: https://svn.code.sf.net/p/jode/code/branches/branch_1_1@1306 379699f6-c40d-0410-875b-85095c16579e

jode/ChangeLog

@@ -1,3 +1,13 @@
+2001-02-05  Jochen Hoenicke  <jochen@gnu.org>
+
+	* jode/expr/InvokeOperator.java.in (Environment.invokeMethod): 
+	Fixed the call to ClassInfo.forName: it expects a class name, while
+	ref.getClazz() gives a type signature.
+	* jode/flow/TransformExceptionHandlers.java.in:
+	(checkAndRemoveJSR): Only invoke removeBadJSR, if there are
+ 	successors left.
+	(checkAndRemoveMonitorExit): likewise.
+
 2001-02-04  Jochen Hoenicke  <jochen@gnu.org>
 
 	* jode/expr/IfThenElseOperator.java (simplify): Allow in the class$

jode/jode/expr/InvokeOperator.java.in

@@ -419,8 +419,10 @@ public final class InvokeOperator extends Operator
 				   Object cls, Object[] params) 
 	    throws InterpreterException, InvocationTargetException {
 	    if (cls == null && ref.getClazz().equals(classSig)) {
-		BytecodeInfo info = 
-		    ClassInfo.forName(ref.getClazz())
+		String clazzName = ref.getClazz();
+		clazzName = clazzName.substring(1, ref.getClazz().length() - 1)
+		    .replace('/', '.');
+		BytecodeInfo info = ClassInfo.forName(clazzName)
 		    .findMethod(ref.getName(), ref.getType())
 		    .getBytecode();
 		if (info != null)

jode/jode/flow/TransformExceptionHandlers.java.in

@@ -387,7 +387,8 @@ public class TransformExceptionHandlers {
 		}
             }
         }
-	removeBadJSR(tryFlow, catchBlock, subRoutine);
+	if (tryFlow.getSuccessors().contains(subRoutine))
+	    removeBadJSR(tryFlow, catchBlock, subRoutine);
     }
 
     private void checkAndRemoveMonitorExit(FlowBlock tryFlow, 
@@ -505,7 +506,8 @@ public class TransformExceptionHandlers {
         }
 
 	if (subRoutine != null) {
-	    removeBadJSR(tryFlow, catchBlock, subRoutine);
+	    if (tryFlow.getSuccessors().contains(subRoutine))
+		removeBadJSR(tryFlow, catchBlock, subRoutine);
 	    tryFlow.mergeAddr(subRoutine);
 	}
     }