Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Yao Qi <yao@codesourcery.com>
To: Pedro Alves <palves@redhat.com>
Cc: Joel Brobecker <brobecker@adacore.com>, <gdb-patches@sourceware.org>
Subject: Re: [PATCH] partial fix PR 14777: Add abbrev cmd 'fo' for 'forward-search'.
Date: Wed, 28 Nov 2012 10:36:00 -0000	[thread overview]
Message-ID: <50B5E90B.9000305@codesourcery.com> (raw)
In-Reply-To: <50B5E4F4.2000905@redhat.com>

On 11/28/2012 06:18 PM, Pedro Alves wrote:
>> +  add_alias_cmd ("tr", "tracepoints", no_class, 1, &deletelist);
> The command being aliased is class_trace, while the alias is no_class.  Why's that?

Because it is a copy-paste error.  How about this one?

-- 
Yao (齐尧)

gdb:

2012-11-28  Yao Qi  <yao@codesourcery.com>

	* breakpoint.c (_initialize_breakpoint) Call add_alias_cmd to
	abbreviate 'delete tracepoints' to 'delete tr'.
	* corefile.c (_initialize_core): Call add_alias_cmd to
	abbreviate 'set gnutarget' to 'set g'.
	* value.c (_initialize_values): Call add_alias_cmd to abbreviate
	'show convenience' to 'show conv'.
---
 gdb/breakpoint.c |    1 +
 gdb/corefile.c   |    1 +
 gdb/value.c      |    1 +
 3 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index 7293d18..3799cbf 100644
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -16339,6 +16339,7 @@ Delete specified tracepoints.\n\
 Arguments are tracepoint numbers, separated by spaces.\n\
 No argument means delete all tracepoints."),
 	   &deletelist);
+  add_alias_cmd ("tr", "tracepoints", class_trace, 1, &deletelist);
 
   c = add_cmd ("tracepoints", class_trace, disable_trace_command, _("\
 Disable specified tracepoints.\n\
diff --git a/gdb/corefile.c b/gdb/corefile.c
index 78dbbc2..eab715d 100644
--- a/gdb/corefile.c
+++ b/gdb/corefile.c
@@ -456,6 +456,7 @@ Use `set gnutarget auto' to specify automatic detection."),
 				   set_gnutarget_command,
 				   show_gnutarget_string,
 				   &setlist, &showlist);
+  add_alias_cmd ("g", "gnutarget", class_files, 1, &setlist);
 
   if (getenv ("GNUTARGET"))
     set_gnutarget (getenv ("GNUTARGET"));
diff --git a/gdb/value.c b/gdb/value.c
index 2a1e1f2..07130ed 100644
--- a/gdb/value.c
+++ b/gdb/value.c
@@ -3384,6 +3384,7 @@ A few convenience variables are given values automatically:\n\
 Convenience functions are defined via the Python API."
 #endif
 	   ), &showlist);
+  add_alias_cmd ("conv", "convenience", no_class, 1, &showlist);
 
   add_cmd ("values", no_set_class, show_values, _("\
 Elements of value history around item number IDX (or last ten)."),
-- 
1.7.7.6


  reply	other threads:[~2012-11-28 10:36 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-07 13:01 Yao Qi
2012-11-07 15:13 ` Joel Brobecker
2012-11-07 16:07   ` Pedro Alves
2012-11-08  9:39   ` Yao Qi
2012-11-08 16:11     ` Pedro Alves
2012-11-28 10:10       ` Yao Qi
2012-11-28 10:18         ` Pedro Alves
2012-11-28 10:36           ` Yao Qi [this message]
2012-11-28 11:10             ` Pedro Alves
2012-11-28 11:57               ` [committed]: " Yao Qi
2012-11-07 16:10 ` Eli Zaretskii
2012-11-09  8:35 ` Yao Qi
2012-11-09  8:43   ` 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=50B5E90B.9000305@codesourcery.com \
    --to=yao@codesourcery.com \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    --cc=palves@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