Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Nick Roberts <nickrob@snap.net.nz>
To: Bob Rossi <bob_rossi@cox.net>
Cc: gdb@sources.redhat.com
Subject: Re: MI query questions
Date: Thu, 01 Jun 2006 00:58:00 -0000	[thread overview]
Message-ID: <17534.15249.56900.580577@kahikatea.snap.net.nz> (raw)
In-Reply-To: <20060531102856.GA29425@brasko.net>

 > >  > > I suggest, for the moment, at least, that we make MI select "[1] all"
 > >  > > automatically in this case.
 > >  > 
 > >  > Nick,
 > >  > 
 > >  > I don't think this solves the problem though. As Daniel pointed out,
 > >  > 
 > >  > -interpreter-exec console "b A::func"
 > >  > 
 > >  > will cause the same problem, and needs to be addressed. I haven't
 > >  > thought this through well enough though.
 > > 
 > > Well, I guess it depends whether the existing behaviour breaks the front
 > > end or not i.e when the prompt ">" appears will it know that GDB wants
 > > more input or not?  If the answer is yes (I've not tried it) then things
 > > can be left as they are.  If its no, then something should be done and the
 > > solution being proposed doesn't sound like a quick fix.
 > > 
 > > "-interpreter-exec console "b A::func" could presumably be made to behave
 > > as "-break-insert A::func".  Witness pending breakpoints:
 > 
 > I wouldn't like to change the behavior of the CLI commands. This would
 > be confusing to users and I don't think they would like it at all.

I don't know how it would work to keep existing CLI behaviour.  I think
"-interpreter-exec console" only works for single isolated CLI commands and
not for ones like "source", "commands", "if", "while" etc or user-defined
ones.  Current behaviour seems to be broken, I'm just trying to suggest an
immediate temporary fix.  Will your solution be ready for 6.5?

 > > (gdb)
 > > -break-insert fgfg
 > > &"Function \"fgfg\" not defined.\n"
 > > ^done
 > > (gdb)
 > > -interpreter-exec console "b ghgh"
 > > &"Function \"ghgh\" not defined.\n"
 > > ~"Breakpoint 1 (ghgh) pending.\n"
 > > ^done
 > > (gdb)
 > > inf bre
 > > &"inf bre\n"
 > > ~"Num Type           Disp Enb Address    What\n"
 > > ~"1   breakpoint     keep y   <PENDING>  ghgh\n"
 > > ^done
 > > (gdb)
 > > 
 > > For the CLI command "break" the behaviour seems to be reversed but it's
 > > no longer a query.
 > 
 > I think this is a bug in GDB that there is no longer a query. The user
 > is going to want to know why they are not prompted when using the FE,
 > but they are prompted when using GDB. I think this is a step backwards,
 > not forwards.

We've had this one before. With "b ghgh", deprecated_query_hook has the value 
mi_interp_query_hook which always returns 1, so this is deliberate.  With
"-break-insert fgfg" the error `Function "fgfg" not defined.' is caught further
up the stack and GDB never gets a chance to set the breakpoint.  I don't
know if that's deliberate.

-- 
Nick                                           http://www.inet.net.nz/~nickrob


      reply	other threads:[~2006-06-01  0:58 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-05-30  3:48 Bob Rossi
2006-05-30  8:20 ` Daniel Jacobowitz
2006-05-30 17:15   ` Jim Ingham
2006-05-30 17:41     ` Bob Rossi
2006-05-30 17:53       ` Jim Ingham
2006-05-30 17:55         ` Jim Ingham
2006-05-30 17:55           ` Bob Rossi
2006-05-30 18:12             ` Daniel Jacobowitz
2006-05-30 20:14             ` Jim Ingham
2006-05-30 18:27               ` Bob Rossi
2006-05-30 18:56                 ` Jim Ingham
2006-05-30 20:46                   ` Bob Rossi
2006-05-30 21:11                     ` Jim Ingham
2006-05-30 21:15                       ` Daniel Jacobowitz
2006-05-30 21:30                         ` Jim Ingham
2006-05-31  9:38                           ` Daniel Jacobowitz
2006-05-31 13:27                             ` Bob Rossi
2006-05-30 17:00 ` Nick Roberts
2006-05-30 17:32   ` Bob Rossi
2006-05-31 10:29     ` Nick Roberts
2006-05-31 13:25       ` Bob Rossi
2006-06-01  0:58         ` Nick Roberts [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=17534.15249.56900.580577@kahikatea.snap.net.nz \
    --to=nickrob@snap.net.nz \
    --cc=bob_rossi@cox.net \
    --cc=gdb@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox