Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Stan Shebs <stan@codesourcery.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Stan Shebs <stan@codesourcery.com>, gdb-patches@sourceware.org
Subject: Re: [PATCH] Fix info tracepoints, etc
Date: Sun, 04 Apr 2010 23:39:00 -0000	[thread overview]
Message-ID: <4BB9231A.5080303@codesourcery.com> (raw)
In-Reply-To: <83y6h5ugum.fsf@gnu.org>

Eli Zaretskii wrote:
>> *************** 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.
>
>   

In this context, the implication is that the user wants to get a list of 
all actionpoints, and it is now the case that only "info break" will 
list everything.  If we wanted to retain a mention of "info watch", 
there would be some complicated rephrasing to clarify that "info watch" 
will not show regular breakpoints etc, and since the point of the 
sentence is just to remind the user that the info command is a 
convenient way to find out the valid actionpoint numbers, simpler seemed 
better.

Stan


      reply	other threads:[~2010-04-04 23:39 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
2010-04-04 23:39   ` Stan Shebs [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=4BB9231A.5080303@codesourcery.com \
    --to=stan@codesourcery.com \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    /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