This is the mail archive of the gdb-patches@sources.redhat.com 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: [RFA] Disable "remote_rdp_can_run"


Richard Earnshaw wrote:
> 
> >
> > Happened across this.  It's not right.
> >
> > With this in place, if you have not attached to your rdp target
> > (ie. by saying "target rdp"), but you instead just say "run",
> > gdb will attempt to use the rdp target, which has not at this
> > point been opened or initialized.  This is not the right way
> > to make a remote target accept the "run" command.
> >
> > 2002-05-02  Michael Snyder  <msnyder@redhat.com>
> >
> >       * remote-rdp.c (remote_rdp_can_run): Return false.  This is
> >       not a good work-around for making a remote target accept 'run'.
> >
> 
> I'm not sure I understand this.  Shouldn't remote_rdp_can_run return 1
> once the target has been attached?  If not, then I think the whole
> function should be killed (so that we pick up the default behaviour).

Actually, I agree with the second statement (it sound be killed).
If you want to have it return true once the target is attached, 
you need some way of detecting that state (perhaps a global).
I didn't bother to do that, because I don't like the idea.

This is the only remote target that tries to do this.
I'd be glad to yank it if you say the word...


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