Hi,

This patch changes VMFrame::getValues to take a signature byte as well as a slot when getting local variables. This is done for use in type checking when interfacing with the VM.

Questions/comments/concerns?

- Kyle


ChangeLog
2007-02-22  Kyle Galloway  <[EMAIL PROTECTED]>

* gnu/classpath/jdwp/processor/StackFrameCommandSet.java (executeGetValues): Pass a signature
   byte to VMFrame.getValue.
* vm/reference/gnu/classpath/jdwp/VMFrame.java (getValue): Add signature parameter.
Index: gnu/classpath/jdwp/processor/StackFrameCommandSet.java
===================================================================
RCS file: /sources/classpath/classpath/gnu/classpath/jdwp/processor/StackFrameCommandSet.java,v
retrieving revision 1.7
diff -u -r1.7 StackFrameCommandSet.java
--- gnu/classpath/jdwp/processor/StackFrameCommandSet.java	2 Feb 2007 14:45:53 -0000	1.7
+++ gnu/classpath/jdwp/processor/StackFrameCommandSet.java	22 Feb 2007 20:14:15 -0000
@@ -115,7 +115,7 @@
       {
         int slot = bb.getInt();
         byte sig = bb.get();
-        Object val = frame.getValue(slot);
+        Object val = frame.getValue(slot, sig);
         Value.writeTaggedValue(os, val);
       }
   }
Index: vm/reference/gnu/classpath/jdwp/VMFrame.java
===================================================================
RCS file: /sources/classpath/classpath/vm/reference/gnu/classpath/jdwp/VMFrame.java,v
retrieving revision 1.6
diff -u -r1.6 VMFrame.java
--- vm/reference/gnu/classpath/jdwp/VMFrame.java	2 Feb 2007 14:45:53 -0000	1.6
+++ vm/reference/gnu/classpath/jdwp/VMFrame.java	22 Feb 2007 20:14:19 -0000
@@ -93,7 +93,7 @@
    * 
    * @param slot the slot containing the variable
    */
-  public native Object getValue(int slot);
+  public native Object getValue(int slot, byte sig);
 
   /**
    * Assigns the given variable to the given value. 

Reply via email to