From: nickrob@snap.net.nz (Nick Roberts)
To: "Marc Khouzam" <marc.khouzam@ericsson.com>
Cc: "Tom Tromey" <tromey@redhat.com>, <gdb-patches@sourceware.org>
Subject: RE: [PATCH] util.c + doc [was Re: [RFC] Queries and frontends]
Date: Thu, 30 Jul 2009 07:24:00 -0000 [thread overview]
Message-ID: <19057.9274.357301.662062@totara.tehura.co.nz> (raw)
In-Reply-To: <6D19CA8D71C89C43A057926FE0D4ADAA07CA09F1@ecamlmw720.eamcs.ericsson.se>
> Doesn't this amount to using "set confirm off"?
> I might have missed it, but I don't remember why you didn't want
> use that instead?
In the console, the user can type "set confirm on/off" according to his
preference. This change ensures that confirmation isn't required for commands
that the frontend runs behind the users back independently of this setting,
provided they are prefixed with "server". This is only for CLI commands and
does not affect frontends which use GDB/MI.
> And this solution does not help (some) frontends with
> answering 'y' to nquery()
nquery will indeed default to 'n'.
> How about:
>
> if (! caution)
> return def_value;
>
> if (server_command)
> return 1;
That might give surprising results, e.g. accidentally setting pending
breakpoints.
AFAIK this change is for _one_ frontend (Emacs) to use _one_ command (record
stop). I think too much generality detracts from this objective as, unlike
GDB/MI, the plan is not to develop annotations.
--
Nick http://www.inet.net.nz/~nickrob
next prev parent reply other threads:[~2009-07-30 4:40 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <6D19CA8D71C89C43A057926FE0D4ADAA07C00023@ecamlmw720.eamcs.ericsson.se>
[not found] ` <19045.23703.743876.775308@totara.tehura.co.nz>
[not found] ` <m37hxwm3wr.fsf@fleche.redhat.com>
[not found] ` <19053.2107.342469.683795@totara.tehura.co.nz>
[not found] ` <m3iqheje2p.fsf@fleche.redhat.com>
[not found] ` <19054.23189.193878.534661@totara.tehura.co.nz>
[not found] ` <m3r5vztcpw.fsf@fleche.redhat.com>
2009-07-30 0:41 ` Nick Roberts
2009-07-30 0:42 ` Marc Khouzam
2009-07-30 7:24 ` Nick Roberts [this message]
2009-07-30 16:02 ` Marc Khouzam
2009-07-30 6:46 ` Eli Zaretskii
2009-08-04 10:12 ` Nick Roberts
2009-08-04 17:51 ` Eli Zaretskii
2009-08-06 5:21 ` Nick Roberts
2009-08-06 8:24 ` Andreas Schwab
2009-08-06 18:11 ` Eli Zaretskii
2009-08-04 16:32 ` [PATCH] util.c + doc Tom Tromey
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=19057.9274.357301.662062@totara.tehura.co.nz \
--to=nickrob@snap.net.nz \
--cc=gdb-patches@sourceware.org \
--cc=marc.khouzam@ericsson.com \
--cc=tromey@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