Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Bob Wilson <bwilson@tensilica.com>
To: Eli Zaretskii <eliz@gnu.org>, Daniel Jacobowitz <drow@false.org>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [doc/commit] Re: fix use of @kbd and @key in gdb.texinfo
Date: Mon, 16 Oct 2006 17:55:00 -0000	[thread overview]
Message-ID: <4533C78B.10800@tensilica.com> (raw)
In-Reply-To: <20061015213827.GA27170@nevyn.them.org>

[-- Attachment #1: Type: text/plain, Size: 1883 bytes --]

Daniel Jacobowitz wrote:
> Bob, please don't do this again; part of the obligation of committing
> "obvious" changes is being responsive when someone disputes their
> obviousness.  Thanks in advance.

Somehow the message that Eli sent directly to me did not get through, probably 
lost in a torrent of spam, and I don't read gdb-patches regularly.  That's not a 
very good excuse, though -- I should have made a point of checking the list in 
case anyone had issues with my change.  I'm sorry.

I will err on the side of caution next time I have a patch that seems obvious.

Eli, thanks for fixing my ChangeLog entry -- I've been writing them that way for 
years and no one has pointed it out to me before.

I'm glad to hear that you want to keep the "Ctrl-" prefix, since it seems more 
clear to me, but I had thought that it was more important to be consistent 
throughout the document.  (My change was motivated by someone reviewing the 
documentation here at Tensilica, who complained about the inconsistency.)  I 
understand what you're saying about using Emacs notation in that section, though.

What about the TUI chapter?  There are lots of "C-" prefixes in there and it's 
not tied to Emacs users.  Should they all use "Ctrl-"?

Also, I noticed that there is a mix of upper- and lowercase for Ctrl key 
combinations, e.g, "Ctrl-C" vs. "Ctrl-c".  Can we make them all the same?  I 
have a slight preference for uppercase, especially because of the "Ctrl-L" case 
that I mentioned earlier, but that goes against the Emacs-style "C-c" convention 
where the letter is lowercase.

Here is a patch to fix one more place to use "Ctrl-".  It was using "C-d" before 
my change, but it seems like it ought to use "Ctrl-" to be consistent, since it 
has nothing to do with Emacs.

2006-10-16  Bob Wilson  <bob.wilson@acm.org>

         * gdb.texinfo (ST2000): Use Ctrl- instead of C-.

[-- Attachment #2: gdb-ctrl.patch --]
[-- Type: text/plain, Size: 658 bytes --]

Index: gdb.texinfo
===================================================================
RCS file: /cvs/src/src/gdb/doc/gdb.texinfo,v
retrieving revision 1.360
diff -u -r1.360 gdb.texinfo
--- gdb.texinfo	15 Oct 2006 21:36:00 -0000	1.360
+++ gdb.texinfo	16 Oct 2006 16:46:19 -0000
@@ -15264,7 +15264,7 @@
 you are done interacting with STDBUG, typing either of two character
 sequences gets you back to the @value{GDBN} command prompt:
 @kbd{@key{RET} ~ .} (Return, followed by tilde and period) or
-@kbd{@key{RET} ~ C-d} (Return, followed by tilde and control-D).
+@kbd{@key{RET} ~ Ctrl-d} (Return, followed by tilde and control-D).
 @end table
 
 @node Z8000

  reply	other threads:[~2006-10-16 17:55 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-10 18:56 Bob Wilson
2006-10-10 21:45 ` Eli Zaretskii
2006-10-15 21:33   ` [doc/commit] " Eli Zaretskii
2006-10-15 21:38     ` Daniel Jacobowitz
2006-10-16 17:55       ` Bob Wilson [this message]
2006-10-17 19:47         ` Eli Zaretskii
2006-10-17 21:18           ` Bob Wilson
2006-10-24 17:52   ` Jim Blandy
2006-10-25  4:30     ` Eli Zaretskii

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=4533C78B.10800@tensilica.com \
    --to=bwilson@tensilica.com \
    --cc=drow@false.org \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sources.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