Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: andre <apoenitz@t-online.de>
To: Joel Brobecker <brobecker@adacore.com>
Cc: Yao Qi <yao@codesourcery.com>, gdb-patches@sourceware.org
Subject: Re: [PATCH] Test no =breakpoint-modified is emitted for modifications from MI commands
Date: Sat, 08 Feb 2014 14:11:00 -0000	[thread overview]
Message-ID: <20140208141122.GA15673@klara.mpi.htwm.de> (raw)
In-Reply-To: <20140208131533.GN5485@adacore.com>

On Sat, Feb 08, 2014 at 05:15:33PM +0400, Joel Brobecker wrote:
> > > One other possible option: Add a new option that would be available
> > > to all commands to disable notifications related to the command being
> > > executed. That way, FE could use it to reduce unnecessary back-chatter.
> > 
> > That is what I am thinking about.

I don't think the amout of information discussed here is worth any special
action on neither gdb's nor a frontend's side. Normal operation produce
"chatter" e.g. for library load/unload notifications on a much bigger
scale.

> > > I don't really like that option, though, as it would require a
> > > certain transition period.
> > 
> > What do you mean by "transition period"?  We can make use of
> > "-list-features" to tell FE that FE can disable/enable MI notifications
> > through a certain command.
> 
> The issue is people using older versions of an FE with a newer version of
> GDB. For those, their FE wouldn't know about the new option and thus get
> the notifications that they might not expect.

This happened regularly with other new notifications in the past
so I would expect frontends to be able to handle new notifications
gracefully. In this case "new" is even relative as the notifications
are sent in most circumstances already anyway.
 
> I don't know if we need to be concerned about this sort of compatibility
> or not...

Since it is not a concern when introducing new notifications like
=cmd-param-changed, =memory-changed or even =breakpoint-modified
itself and frontends need to handle the case of "unexpected" 
notifications anyway, it's hard for me to see how sending a specific
notification in all cases instead of in "most" cases can do harm.

Andre'


  reply	other threads:[~2014-02-08 14:11 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-24  7:48 Yao Qi
2014-02-06 12:38 ` Yao Qi
2014-02-06 20:39   ` andre
2014-02-07  9:14     ` Yao Qi
2014-02-07 16:12       ` andre
2014-02-08  3:18         ` Joel Brobecker
2014-02-08  3:39           ` Yao Qi
2014-02-08 13:15             ` Joel Brobecker
2014-02-08 14:11               ` andre [this message]
2014-02-07 14:04 ` Pedro Alves
2014-02-08  1:49   ` Yao Qi

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=20140208141122.GA15673@klara.mpi.htwm.de \
    --to=apoenitz@t-online.de \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    --cc=yao@codesourcery.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