This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] fix to_open debug setting
- From: Tom Tromey <tromey at redhat dot com>
- To: gdb-patches at sourceware dot org
- Date: Tue, 22 Jul 2014 09:27:01 -0600
- Subject: Re: [PATCH] fix to_open debug setting
- Authentication-results: sourceware.org; auth=none
- References: <1405975154-19517-1-git-send-email-tromey at redhat dot com> <87ha2as33y dot fsf at fleche dot redhat dot com>
Tom> Later I happened to notice that add_deprecated_target_alias could use
Tom> the same treatment. I'll send the new patch tomorrow.
Here it is.
Tom
2014-07-21 Tom Tromey <tromey@redhat.com>
* target.c (open_target): New function.
(add_target_with_completer, add_deprecated_target_alias): Use
set_cmd_sfunc, set_cmd_context.
(debug_to_open): Remove.
(setup_target_debug): Update.
diff --git a/gdb/target.c b/gdb/target.c
index b9310cf..42d7800 100644
--- a/gdb/target.c
+++ b/gdb/target.c
@@ -118,8 +118,6 @@ static struct target_ops debug_target;
static void init_dummy_target (void);
-static void debug_to_open (char *, int);
-
/* Pointer to array of target architecture structures; the size of the
array; the current index into the array; the allocated size of the
array. */
@@ -345,6 +343,19 @@ complete_target_initialization (struct target_ops *t)
install_delegators (t);
}
+/* This is used to implement the various target commands. */
+
+static void
+open_target (char *args, int from_tty, struct cmd_list_element *command)
+{
+ struct target_ops *ops = get_cmd_context (command);
+
+ if (targetdebug)
+ fprintf_unfiltered (gdb_stdlog, "target_open (%s, %d)\n", args, from_tty);
+
+ ops->to_open (args, from_tty);
+}
+
/* Add possible target architecture T to the list and add a new
command 'target T->to_shortname'. Set COMPLETER as the command's
completer if not NULL. */
@@ -380,8 +391,9 @@ Remaining arguments are interpreted by the target protocol. For more\n\
information on the arguments for a particular protocol, type\n\
`help target ' followed by the protocol name."),
&targetlist, "target ", 0, &cmdlist);
- c = add_cmd (t->to_shortname, no_class, t->to_open, t->to_doc,
- &targetlist);
+ c = add_cmd (t->to_shortname, no_class, NULL, t->to_doc, &targetlist);
+ set_cmd_sfunc (c, open_target);
+ set_cmd_context (c, t);
if (completer != NULL)
set_cmd_completer (c, completer);
}
@@ -404,7 +416,9 @@ add_deprecated_target_alias (struct target_ops *t, char *alias)
/* If we use add_alias_cmd, here, we do not get the deprecated warning,
see PR cli/15104. */
- c = add_cmd (alias, no_class, t->to_open, t->to_doc, &targetlist);
+ c = add_cmd (alias, no_class, NULL, t->to_doc, &targetlist);
+ set_cmd_sfunc (c, open_target);
+ set_cmd_context (c, t);
alt = xstrprintf ("target %s", t->to_shortname);
deprecate_cmd (c, alt);
}
@@ -2946,13 +2960,6 @@ init_dummy_target (void)
install_dummy_methods (&dummy_target);
}
-static void
-debug_to_open (char *args, int from_tty)
-{
- debug_target.to_open (args, from_tty);
-
- fprintf_unfiltered (gdb_stdlog, "target_open (%s, %d)\n", args, from_tty);
-}
void
target_close (struct target_ops *targ)
@@ -3397,7 +3404,6 @@ setup_target_debug (void)
{
memcpy (&debug_target, ¤t_target, sizeof debug_target);
- current_target.to_open = debug_to_open;
init_debug_target (¤t_target);
}