From f664381a5d59bef23b77ab49b2634bd0a36be581 Mon Sep 17 00:00:00 2001 From: kseitz Date: Mon, 25 Jan 2010 19:59:29 +0000 Subject: [PATCH] * generic/gdbtk-cmds.c (gdb_set_inferior_args): set_inferior_args now works as expected. Do not strdup/free the argument string. --- gdb/gdbtk/ChangeLog | 5 +++++ gdb/gdbtk/generic/gdbtk-cmds.c | 8 +------- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/gdb/gdbtk/ChangeLog b/gdb/gdbtk/ChangeLog index 0fb5ac20d0..a3138887ff 100644 --- a/gdb/gdbtk/ChangeLog +++ b/gdb/gdbtk/ChangeLog @@ -1,3 +1,8 @@ +2010-01-25 Keith Seitz + + * generic/gdbtk-cmds.c (gdb_set_inferior_args): set_inferior_args + now works as expected. Do not strdup/free the argument string. + 2010-01-15 Stan Shebs * generic/gdbtk-bp.c (gdb_trace_status): Use current_trace_status. diff --git a/gdb/gdbtk/generic/gdbtk-cmds.c b/gdb/gdbtk/generic/gdbtk-cmds.c index 20ead8d4c8..6a249f5ae5 100644 --- a/gdb/gdbtk/generic/gdbtk-cmds.c +++ b/gdb/gdbtk/generic/gdbtk-cmds.c @@ -859,13 +859,7 @@ gdb_set_inferior_args (ClientData clientData, Tcl_Interp *interp, } args = Tcl_GetStringFromObj (objv[1], NULL); - - /* The xstrdup/xfree stuff is so that we maintain a coherent picture - for gdb. I would expect the accessors to do this, but they - don't. */ - args = xstrdup (args); - args = set_inferior_args (args); - xfree (args); + set_inferior_args (args); return TCL_OK; } -- 2.11.0