Re: insight/52

[Fernando Nasser <fnasser at cygnus dot com>]

Steven Johnson wrote:
> 
> I Had to change this to read
>        if {$gdb_running} {
>          return
>        }
> 
> Otherwise the TCL Interpreter hated it.
> 
Thanks, as I said, I had no way of testing it :)


> Did this, and download worked great. BUT...
> 
> Now I can Connect, and Download OK.
> 
> I Can insert a breakpoint and RUN OK.
> 
> If I then insert another breakpoint and CONTINUE.
> 
> The target runs and breaks, but Insight doesn't show the break having occured.
> I can change the source file and come back to the file it is broken in and the
> state changes for "GDB Running" to "GDB Halted" but the indicated line is
> wrong. (I can view the registers and the $pc is definately pointing to the next
> breakpoint instruction.)
> 
> I think it is something to do with the change we made to gdbtk_update, but i'm
> not sure what.
> 

Yeah, I was afraid that would happen but I hoped the gdb_running was going to be 
unset before gdbtk_update.

I will have to create a "safe_gdbtk_update" for the after commands and leave the
original alone.

I am sending you the patch momentarily.




-- 
Fernando Nasser
Red Hat Canada Ltd.                     E-Mail:  fnasser@cygnus.com
2323 Yonge Street, Suite #300
Toronto, Ontario   M4P 2C9