Re: [PATCH] Add shortcuts for "nexti" and "stepi" commands in Single-Key mode

Ruslan Kabatsayev
2017-07-13T18:19:17Z

On 13/07/17 18:59, Eli Zaretskii wrote:
>> From: Ruslan Kabatsayev <b7.10110111@gmail.com>
>> Cc: Ruslan Kabatsayev <b7.10110111@gmail.com>
>> Date: Thu, 13 Jul 2017 09:07:50 +0300
>>
>> Currently, "layout asm" is not so useful as "layout src" with Single-Key mode:
>> you have to use multi-key commands like "ni" and "si" to do single-stepping.
>> This patch adds, in addition to "next" and "step" commands, corresponding
>> assembly-level ones - "nexti" and "stepi" - to Single-Key mode, with the
>> shortcuts of "N" and "S" respectively.
>>
>> gdb/ChangeLog:
>>
>> * tui/tui.c: Add "nexti" and "stepi" to the tui_commands list
>> * doc/gdb.texinfo: Document the new shortcuts in Single-Key mode
>
> This is fine with me, but please state in the ChangeLog the
> functions/globals where you make the changes, and for gdb.texinfo
> please state the name of the node (in parentheses). I think the
> existing ChangeLog entries show how we do that.

gdb/ChangeLog:
* tui/tui.c (tui_commands): Add "nexti" and "stepi" to the tui_commands list
* doc/gdb.texinfo (TUI Single Key Mode): Document the new shortcuts in
Single-Key mode

>
> Thanks.
>