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

Pedro Alves
2012-11-28T10:18:51Z

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