From: Eli Zaretskii <eliz@gnu.org>
To: Stan Shebs <stan@codesourcery.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH] Fix info tracepoints, etc
Date: Sat, 03 Apr 2010 07:38:00 -0000 [thread overview]
Message-ID: <83y6h5ugum.fsf@gnu.org> (raw)
In-Reply-To: <4BB6AF76.6020101@codesourcery.com>
> Date: Fri, 02 Apr 2010 20:01:10 -0700
> From: Stan Shebs <stan@codesourcery.com>
>
> Originally just a simple fix to make "info tracepoints" display only
> tracepoints, it's grown a bit. :-) The key change is to pass an optional
> filter function to breakpoint_1, which has a nice bonus of letting "info
> watchpoints" being the only-watchpoints lister that it was always
> supposed to be - which then requires doc and testsuite changes. I also
> took the opportunity of merging the duplicate tracepoint predicates that
> found their way into the code, plus there's now a warning that
> tracepoints don't do ignore counts.
Thanks.
> *** doc/gdb.texinfo 1 Apr 2010 14:11:23 -0000 1.693
> --- doc/gdb.texinfo 3 Apr 2010 02:48:16 -0000
This part is okay, but I have one question:
> *************** it had been deleted, but remembers the i
> *** 4135,4144 ****
> that you can @dfn{enable} it again later.
>
> You disable and enable breakpoints, watchpoints, and catchpoints with
> ! the @code{enable} and @code{disable} commands, optionally specifying one
> ! or more breakpoint numbers as arguments. Use @code{info break} or
> ! @code{info watch} to print a list of breakpoints, watchpoints, and
> ! catchpoints if you do not know which numbers to use.
>
> Disabling and enabling a breakpoint that has multiple locations
> affects all of its locations.
> --- 4134,4143 ----
> that you can @dfn{enable} it again later.
>
> You disable and enable breakpoints, watchpoints, and catchpoints with
> ! the @code{enable} and @code{disable} commands, optionally specifying
> ! one or more breakpoint numbers as arguments. Use @code{info break} to
> ! print a list of all breakpoints, watchpoints, and catchpoints if you
> ! do not know which numbers to use.
Why did you need to remove the reference to "info watch" from this
passage? The command is still available, and will do its job.
next prev parent reply other threads:[~2010-04-03 7:38 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-03 3:01 Stan Shebs
2010-04-03 7:38 ` Eli Zaretskii [this message]
2010-04-04 23:39 ` Stan Shebs
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=83y6h5ugum.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=gdb-patches@sourceware.org \
--cc=stan@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