* generic/gdbtk-cmds.c (gdb_set_inferior_args): set_inferior_args

	now works as expected. Do not strdup/free the argument string.

diff --git a/gdb/gdbtk/ChangeLog b/gdb/gdbtk/ChangeLog
index 0fb5ac2..a313888 100644 (file)
--- a/gdb/gdbtk/ChangeLog
+++ b/gdb/gdbtk/ChangeLog
@@ -1,3 +1,8 @@
+2010-01-25  Keith Seitz  <keiths@redhat.com>
+
+       * 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  <stan@codesourcery.com>
 
        * 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 20ead8d..6a249f5 100644 (file)
--- 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;
 }