Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Tom Tromey <tromey@redhat.com>
To: Pedro Alves <palves@redhat.com>
Cc: Sergio Durigan Junior <sergiodj@redhat.com>,
	       Pierre Muller <pierre.muller@ics-cnrs.unistra.fr>,
	       "'GDB Patches'" <gdb-patches@sourceware.org>
Subject: Re: [RFC/PATCH] New convenience variable $_exitsignal
Date: Tue, 17 Sep 2013 18:39:00 -0000	[thread overview]
Message-ID: <87bo3rxpko.fsf@fleche.redhat.com> (raw)
In-Reply-To: <52374823.4010203@redhat.com> (Pedro Alves's message of "Mon, 16	Sep 2013 19:04:19 +0100")

>>>>> "Pedro" == Pedro Alves <palves@redhat.com> writes:

Pedro> I can't say I really understand how any of that argues against my
Pedro> original rationale for not setting $_exitsignal on corefiles (because
Pedro> the inferior has not really exited at the point the core has been
Pedro> generated), rather than point at implementation choices.

Pedro> Now, if one were to instead argue that _user interface_ -wise, it'd
Pedro> make sense to set $_exitsignal, because we also print
Pedro> "Program terminated with signal", (emphasis on "terminated"), then
Pedro> I'd agree:

I may have missed part of the thread, but what is the rationale for
being so precise here?  It seems a bit pedantic to me -- which is fine
if there is a purpose to it, but I couldn't think of one in this case.

That is, gdb has a bit of information that is relevant to the user.  It
is useful to users if we expose it to them in a script-friendly way.  We
already have $_exitsignal, and differentiating between "the program
exited with signal X" and "the program is about to exit with signal X"
doesn't seem particularly handy.

Another consideration along these lines is that I have a branch in
progress for "catch exit" -- it's been waiting for Sergio's work on
these convenience variables.  I think here as well $_exitsignal seems
like a natural fit, even though the process has not technically exited
at the catchpoint.

Tom


  parent reply	other threads:[~2013-09-17 18:39 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-16  6:30 Sergio Durigan Junior
2013-06-16 16:22 ` Doug Evans
2013-06-17  3:33 ` Eli Zaretskii
2013-06-17  7:32 ` Pierre Muller
2013-06-17 17:55   ` Sergio Durigan Junior
2013-06-19  5:26     ` Sergio Durigan Junior
2013-09-16 18:04       ` Pedro Alves
2013-09-17  0:11         ` Sergio Durigan Junior
2013-09-17 16:19           ` Pedro Alves
2013-09-17 18:39         ` Tom Tromey [this message]
2013-09-17 18:53           ` Philippe Waroquiers
2013-09-17 18:59             ` Tom Tromey
2013-09-17 18:59             ` Sergio Durigan Junior
2013-09-17 19:08             ` Pedro Alves
2013-09-17 19:02           ` Pedro Alves
2013-09-17 19:09             ` Tom Tromey
2013-07-18 16:48     ` Tom Tromey
2013-06-17 17:28 ` Pedro Alves
2013-06-17 17:31   ` Pedro Alves
2013-06-17 17:41   ` Sergio Durigan Junior

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=87bo3rxpko.fsf@fleche.redhat.com \
    --to=tromey@redhat.com \
    --cc=gdb-patches@sourceware.org \
    --cc=palves@redhat.com \
    --cc=pierre.muller@ics-cnrs.unistra.fr \
    --cc=sergiodj@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