diff options
| author | Simon Marchi <simon.marchi@polymtl.ca> | 2021-05-17 14:01:14 -0400 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@polymtl.ca> | 2021-05-17 14:01:14 -0400 |
| commit | 9985872497e2b8c86424fcb97cd9a065f406a5c5 (patch) | |
| tree | a293dbd7159ddf4295a39381da29a33ab9a5b8da /gdb/unittests | |
| parent | 14b42fc4a0edc412e02a6c9cfe7eb48e67da3145 (diff) | |
gdb: rename cmd_list_element::cmd_pointer to target
cmd_pointer is another field whose name I found really not clear. Yes,
it's a pointer to a command, the type tells me that. But what's the
relationship of that command to the current command? This field
contains, for an alias, the command that it aliases. So I think that
the name "alias_target" would be more appropriate.
Also, rename "old" parameters to "target" in the functions that add
aliases.
gdb/ChangeLog:
* cli/cli-decode.h (cmd_list_element) <cmd_pointer>: Rename
to...
<alias_target>: ... this.
(add_alias_cmd): Rename old to target.
(add_info_alias): Rename old_name to target_name.
(add_com_alias): Likewise.
Change-Id: I8db36c6dd799fae155f7acd3805f6d62d98befa9
Diffstat (limited to 'gdb/unittests')
| -rw-r--r-- | gdb/unittests/command-def-selftests.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/unittests/command-def-selftests.c b/gdb/unittests/command-def-selftests.c index 53e5626f64..123667d336 100644 --- a/gdb/unittests/command-def-selftests.c +++ b/gdb/unittests/command-def-selftests.c @@ -155,7 +155,7 @@ traverse_command_structure (struct cmd_list_element **list, { /* If this command has subcommands and is not an alias, traverse the subcommands. */ - if (c->subcommands != NULL && c->cmd_pointer == nullptr) + if (c->subcommands != NULL && c->alias_target == nullptr) { /* Recursively call ourselves on the subcommand list, passing the right prefix in. */ |