Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Philippe Waroquiers <philippe.waroquiers@skynet.be>
To: Pedro Alves <palves@redhat.com>,
	Kevin Buettner <kevinb@redhat.com>,
		gdb@sourceware.org
Cc: John Baldwin <jhb@FreeBSD.org>
Subject: Re: / (slash) command : fast way to temporarily change setting(s) to run a command
Date: Fri, 03 May 2019 09:10:00 -0000	[thread overview]
Message-ID: <1556874602.1511.10.camel@skynet.be> (raw)
In-Reply-To: <d96f6319-0651-6765-321f-476667c79cea@redhat.com>

On Fri, 2019-05-03 at 09:23 +0100, Pedro Alves wrote:
> > 
> > I like this rationale for the global / command.
> > 
> > I'd like to wait a bit longer before reviewing Philippe's patch, however.
> > I know that one of my colleagues has some thoughts on the interface
> > and I'd like to give him a little extra time to express them here...
> 
> That was me.  :-)  See here:
> 
>  https://sourceware.org/ml/gdb-patches/2019-05/msg00002.html
> 
> I'll follow up when I have a chance.  Yesterday I fixed the
> regressions that Philippe pointed at (fixes already in the branch) -- I
> wanted to see if it was possible to get to a clean slate for better
> experimenting / better ground for discussion.
Thanks for this.
Note that  I have listed some points for the discussion in
https://sourceware.org/ml/gdb-patches/2019-05/msg00019.html

IMO, a (the?) major question I still have about keeping (or not) the "/"
patch is for options that have a 'wide' impact.
The "/" command avoids that this option has to be added as a specific
argument to all the commands that are impacted directly or indirectly.
An example is the new max-depth.
As far as I understand, max-depth will at least impact:
   backtrace
   print
   frame
   info args
   ...
(in fact all commands that are printing directly or indirectly
some variables).
With the "/" command, you have a way to influence all these commands.
Otherwise, as far as I understand, an option --max-depth has
to be added to all these commands.
(or the user has to do set/launch command/reset).

Philippe



      reply	other threads:[~2019-05-03  9:10 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1555313564.1473.2.camel@skynet.be>
2019-04-16  3:06 ` Simon Marchi
2019-04-16  6:48   ` Philippe Waroquiers
2019-04-16 12:20     ` Simon Marchi
2019-04-16 21:22       ` Philippe Waroquiers
2019-04-17  8:14 ` Kevin Buettner
2019-04-17 20:59   ` Philippe Waroquiers
2019-04-19 16:52   ` John Baldwin
2019-05-03  3:10     ` Kevin Buettner
2019-05-03  8:23       ` Pedro Alves
2019-05-03  9:10         ` Philippe Waroquiers [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=1556874602.1511.10.camel@skynet.be \
    --to=philippe.waroquiers@skynet.be \
    --cc=gdb@sourceware.org \
    --cc=jhb@FreeBSD.org \
    --cc=kevinb@redhat.com \
    --cc=palves@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