[RFA] JDWP implement VMVM::getFrame

Tom Tromey tromey@redhat.com
Wed Feb 7 23:15:00 GMT 2007


>>>>> "Kyle" == Kyle Galloway <kgallowa@redhat.com> writes:

Kyle> This patch implements the JDWP method getFrame from
Kyle> natVMVirtualMachine.cc.

Looks good.  Just one nit...

Kyle> +    loc = new Location (meth, static_cast<jlong> (info.location));

By my reading, info.location is a jlong, so we shouldn't need this
cast.

Assuming I'm right, this is ok with that change.
And if I'm wrong, it is ok without it :)

Tom



More information about the Java-patches mailing list