This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA/JDWP] Implement VMVM:getFrames


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

Kyle> This patch implements the final portion of JDWP stack tracing.  what I
Kyle> have done here is to use VMVM:getFrame to get each frame and them just
Kyle> pack them into an array.  It is also possible to have getFrames call
Kyle> JVMTI directly, but it think it would be better to have getFrame do
Kyle> the processing from JVMTI to a VMFrame to avoid having to duplicate
Kyle> much of that code.  Are there other opinions?

If this approach is ok with Keith, it is ok with me.

Kyle> +      // Only return length frames.
Kyle> +      if (length != -1 && length-- <= 0)
Kyle> +        break; 

Why?

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]