This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] partial fix PR 14777: Add abbrev cmd 'fo' for 'forward-search'.


On 11/28/2012 10:09 AM, Yao Qi wrote:
> On 11/09/2012 12:10 AM, Pedro Alves wrote:
>> These are user visible changes, and if we don't tell users about the
>> change in NEWS, those who have tripped on the docs-not-match-reality
>> issue before will never try the abbreviations again, thinking they
>> don't really work, even though they do now work.  IOW, IMO, this
>> is NEWS worty.
> 
> Pedro, changes in this patch [1] are not user visible, because even without this patch nowadays, the (implicitly defined) abbreviated commands still behave correctly.  This patch [1] is to prevent these abbreviated commands becoming ambiguous in the future.

I see.

> Of course, this is a ping to the patch [1].

> --- a/gdb/breakpoint.c
> +++ b/gdb/breakpoint.c
> @@ -16337,6 +16337,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", no_class, 1, &deletelist);

The command being aliased is class_trace, while the alias is no_class.  Why's that?

-- 
Pedro Alves


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]