Message

Reply | Reply all

Date Prev | Date Next | Thread Prev | Thread Next Date Index | Thread Index

From: cvs at opencores.org<cvs@o...>
Date: Thu Nov 29 16:03:38 CET 2007
Subject: [cvs-checkins] MODIFIED: jop ...
Date: 00/07/11 29:16:03

Modified: jop/java/tools/src/com/jopdesign/debug/jdwp MethodTable.java
Log:
Fixed bug in the method ID assignment.



Methods changed:

78: public int addMethod(MethodInfo method)

- Fixed subtle bug: the field used as ID should be the

address of the method structure, not from the method code.





Methods created:

98: public static int getMethodId(MethodInfo methodInfo)




Revision Changes Path
1.2 jop/java/tools/src/com/jopdesign/debug/jdwp/MethodTable.java

http://www.opencores.org/cvsweb.shtml/jop/java/tools/src/com/jopdesign/debug/jdwp/MethodTable.java.diff?r1=1.1&r2=1.2

(In the diff below, changes in quantity of whitespace are not shown.)

Index: MethodTable.java
===================================================================
RCS file: /cvsroot/paulo/jop/java/tools/src/com/jopdesign/debug/jdwp/MethodTable.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -b -r1.1 -r1.2
--- MethodTable.java 3 Oct 2007 00:50:33 -0000 1.1
+++ MethodTable.java 29 Nov 2007 15:03:37 -0000 1.2
@@ -77,7 +77,7 @@
*/
public int addMethod(MethodInfo method)
{
- int methodId = method.getCodeAddress();
+ int methodId = getMethodId(method);
if(isValidIndex(methodId) == false)
{
// increase the size so it can hold the new element.
@@ -88,4 +88,16 @@

return methodId;
}
+
+ /**
+ * Return the method ID.
+ *
+ * @param methodInfo
+ * @return
+ */
+ public static int getMethodId(MethodInfo methodInfo)
+ {
+ //return methodInfo.getCodeAddress();
+ return methodInfo.getStructAddress();
+ }
}