diff options
| author | Tom Tromey <tom@tromey.com> | 2017-09-09 20:51:33 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2017-09-27 08:44:14 -0600 |
| commit | 0450cc4ce8b4775c47e9aaa1c5a34e181b10ae98 (patch) | |
| tree | f72a3f6796ac10fe11c3ca341aaf22aea3a8b693 /gdb/guile/scm-cmd.c | |
| parent | a9bbfbd85fddfea2db30810c33c4bb130a2ef773 (diff) | |
| download | gdb-0450cc4ce8b4775c47e9aaa1c5a34e181b10ae98.zip gdb-0450cc4ce8b4775c47e9aaa1c5a34e181b10ae98.tar.gz gdb-0450cc4ce8b4775c47e9aaa1c5a34e181b10ae98.tar.bz2 | |
Add add_cmd function overloads
This adds two add_cmd overloads: one whose callback takes a const char *,
and one that doesn't accept a function at all. The no-function overload
was introduced to avoid ambiguity when NULL was passed as the function.
Long term the goal is for all commands to take const arguments, and
for the non-const variants to be removed entirely.
gdb/ChangeLog
2017-09-27 Tom Tromey <tom@tromey.com>
* cli/cli-decode.c (add_cmd, set_cmd_cfunc): New function
overloads.
(do_add_cmd): Rename from add_cmd. Don't call set_cmd_cfunc.
(do_const_cfunc): New function.
(cmd_cfunc_eq): New overload.
(cli_user_command_p): Check do_const_cfunc.
* cli/cli-decode.h (struct cmd_list_element) <function>: New field
const_cfunc.
* command.h (add_cmd): Add const overload and no-function
overload.
(set_cmd_cfunc): Add const overload.
(cmd_const_cfunc_ftype): Declare.
(cmd_cfunc_eq): Add const overload.
* breakpoint.c, cli-cmds.c, cli-dump.c, guile/scm-cmd.c,
python/py-cmd.c, target.c, tracepoint.c: Use no-function add_cmd
overload.
Diffstat (limited to 'gdb/guile/scm-cmd.c')
| -rw-r--r-- | gdb/guile/scm-cmd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/guile/scm-cmd.c b/gdb/guile/scm-cmd.c index 5501d31..0bd5105 100644 --- a/gdb/guile/scm-cmd.c +++ b/gdb/guile/scm-cmd.c @@ -774,7 +774,7 @@ gdbscm_register_command_x (SCM self) else { cmd = add_cmd (c_smob->cmd_name, c_smob->cmd_class, - NULL, c_smob->doc, cmd_list); + c_smob->doc, cmd_list); } } CATCH (except, RETURN_MASK_ALL) |