Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Pedro Alves <pedro@codesourcery.com>
To: gdb-patches@sourceware.org
Cc: Michael Snyder <msnyder@vmware.com>,
	 "tromey@redhat.com" <tromey@redhat.com>
Subject: Re: PR8554: New command to save breakpoints to a file
Date: Thu, 15 Apr 2010 19:58:00 -0000	[thread overview]
Message-ID: <201004152058.06658.pedro@codesourcery.com> (raw)
In-Reply-To: <4BC761E4.2040505@vmware.com>

On Thursday 15 April 2010 19:58:44, Michael Snyder wrote:
> >>> OTOH, it may be useful to be able to dump watchpoints on locals,
> >>> and be able to load them up when you know it's okay, so never
> >>> dumping those isn't that great either.  So, IMO, we shouldn't
> >>> worry much about those, at least, in this first patch.  :-)  I
> >>> could add a note to the manual, perhaps.
> >> That's cool.  So what do we do now?  Just skip them?
> >> Save the global ones, skip the local ones?
> > 
> > We save them.
> > 
> 
> Oh -- and on reload, some of them fail?
> 
> Shouldn't be difficult to skip saving all of them, but what about
> skipping the locals and saving the globals?  Hard?

As I said above, it may be useful to be able to dump watchpoints
on locals, and be able to load them up when you know it's okay,
so never dumping those isn't that great either.  It's not hard,
it's just not always the right thing.  As is, the user can edit the
script if the wants to zap some breakpoints before sourcing it.  It's
just a CLI script.  If you want to add a new command switch to
tune the behaviour, that'd be cool.

-- 
Pedro Alves


  reply	other threads:[~2010-04-15 19:58 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-09  2:41 Pedro Alves
2010-04-09 16:17 ` Tom Tromey
2010-04-09 17:23   ` Pedro Alves
2010-04-10  4:10     ` Sergio Durigan Junior
2010-04-12 18:15     ` Pedro Alves
2010-04-15 17:52       ` Michael Snyder
2010-04-15 18:27         ` Pedro Alves
2010-04-15 18:46           ` Michael Snyder
2010-04-15 18:53             ` Pedro Alves
2010-04-15 18:58               ` Michael Snyder
2010-04-15 19:58                 ` Pedro Alves [this message]
2010-04-15 22:49                   ` Michael Snyder
2010-04-15 22:52                     ` Michael Snyder
2010-04-15 22:58                       ` Michael Snyder
2010-04-15 23:05                         ` Pedro Alves
2010-04-15 23:15                           ` Michael Snyder
2010-04-15 23:25                             ` Pedro Alves
2010-05-05  8:28                               ` Hui Zhu
2010-04-15 23:14                         ` Michael Snyder
2010-04-15 23:20                           ` Pedro Alves
2010-04-15 23:01                     ` Pedro Alves
2010-04-15 18:59           ` Stan Shebs
2010-04-15 19:10             ` Pedro Alves

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=201004152058.06658.pedro@codesourcery.com \
    --to=pedro@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    --cc=msnyder@vmware.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