This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB 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: [PATCH] Add temporary frames to frame cache.


On 07/06/2013 8:18 PM, Pedro Alves wrote:
> 
> (BTW, I find the fact that "frame NUMBER" can create a new frame to
> be really a bad interface, and very confusing for users.  

I agree, I only spotted this issue as I miss-typed the frame number I
really wanted to change to and was surprised I didn't get an error..

>                                                              I'd
> much rather the command would error out if the frame is not found
> (as most frequently that'll be a user mistake), and the user that
> really knows what she's doing and wants to inspect stack at
> some address, would instead do something like
> "frame -create ADDRESS" to create a new frame at a random address.
> The implementation of that would indeed then install this new
> frame as current_frame, and so "bt" from there would start
> unwinding from ADDRESS.  The documentation could then also
> document how to get out of this frame in order to start unwinding
> from the current registers again.  E.g., pointing at "flushregs".)

I think that would be much better, and I'm happy to put together a patch
to do this unless someone disagrees.

Thanks,
Andrew



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