Fix crash when a variable object being deleted
[deliverable/binutils-gdb.git] / gdb / interps.c
index 2a4cc0b52f6737eb3a6143aa433a997af6a0ec9d..3b6b4158195f3ed6dc4094e18e479e44a7fbf429 100644 (file)
@@ -1,6 +1,6 @@
 /* Manages interpreters for GDB, the GNU debugger.
 
-   Copyright 2000, 2002, 2003 Free Software Foundation, Inc.
+   Copyright (C) 2000, 2002, 2003, 2007 Free Software Foundation, Inc.
 
    Written by Jim Ingham <jingham@apple.com> of Apple Computer, Inc.
 
@@ -8,7 +8,7 @@
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
-   the Free Software Foundation; either version 2 of the License, or
+   the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
 
    This program is distributed in the hope that it will be useful,
@@ -17,9 +17,7 @@
    GNU General Public License for more details.
 
    You should have received a copy of the GNU General Public License
-   along with this program; if not, write to the Free Software
-   Foundation, Inc., 59 Temple Place - Suite 330,
-   Boston, MA 02111-1307, USA. */
+   along with this program.  If not, see <http://www.gnu.org/licenses/>. */
 
 /* This is just a first cut at separating out the "interpreter"
    functions of gdb into self-contained modules.  There are a couple
@@ -42,6 +40,7 @@
 #include "gdb-events.h"
 #include "gdb_assert.h"
 #include "top.h"               /* For command_loop.  */
+#include "exceptions.h"
 
 struct interp
 {
@@ -143,7 +142,7 @@ interp_set (struct interp *interp)
          && !current_interpreter->procs->suspend_proc (current_interpreter->
                                                        data))
        {
-         error ("Could not suspend interpreter \"%s\"\n",
+         error (_("Could not suspend interpreter \"%s\"."),
                 current_interpreter->name);
        }
     }
@@ -177,7 +176,7 @@ interp_set (struct interp *interp)
       interp->inited = 1;
     }
 
-  /* Clear out any installed interpreter hooks/event handlers. */
+  /* Clear out any installed interpreter hooks/event handlers.  */
   clear_interpreter_hooks ();
 
   if (interp->procs->resume_proc != NULL
@@ -185,7 +184,7 @@ interp_set (struct interp *interp)
     {
       if (old_interp == NULL || !interp_set (old_interp))
        internal_error (__FILE__, __LINE__,
-                       "Failed to initialize new interp \"%s\" %s",
+                       _("Failed to initialize new interp \"%s\" %s"),
                        interp->name, "and could not restore old interp!\n");
       return 0;
     }
@@ -304,19 +303,18 @@ interp_exec_p (struct interp *interp)
   return interp->procs->exec_proc != NULL;
 }
 
-int
+struct gdb_exception
 interp_exec (struct interp *interp, const char *command_str)
 {
   if (interp->procs->exec_proc != NULL)
     {
       return interp->procs->exec_proc (interp->data, command_str);
     }
-  return 0;
+  return exception_none;
 }
 
-/* A convenience routine that nulls out all the
-   common command hooks.  Use it when removing your interpreter in its 
-   suspend proc. */
+/* A convenience routine that nulls out all the common command hooks.
+   Use it when removing your interpreter in its suspend proc.  */
 void
 clear_interpreter_hooks (void)
 {
@@ -329,7 +327,6 @@ clear_interpreter_hooks (void)
   deprecated_delete_breakpoint_hook = 0;
   deprecated_modify_breakpoint_hook = 0;
   deprecated_interactive_hook = 0;
-  deprecated_registers_changed_hook = 0;
   deprecated_readline_begin_hook = 0;
   deprecated_readline_hook = 0;
   deprecated_readline_end_hook = 0;
@@ -344,10 +341,10 @@ clear_interpreter_hooks (void)
   clear_gdb_event_hooks ();
 }
 
-/* This is a lazy init routine, called the first time
-   the interpreter module is used.  I put it here just in case, but I haven't
-   thought of a use for it yet.  I will probably bag it soon, since I don't
-   think it will be necessary. */
+/* This is a lazy init routine, called the first time the interpreter
+   module is used.  I put it here just in case, but I haven't thought
+   of a use for it yet.  I will probably bag it soon, since I don't
+   think it will be necessary.  */
 static void
 initialize_interps (void)
 {
@@ -368,7 +365,7 @@ interpreter_exec_cmd (char *args, int from_tty)
   prules = buildargv (args);
   if (prules == NULL)
     {
-      error ("unable to parse arguments");
+      error (_("unable to parse arguments"));
     }
 
   nrules = 0;
@@ -381,29 +378,30 @@ interpreter_exec_cmd (char *args, int from_tty)
     }
 
   if (nrules < 2)
-    error ("usage: interpreter-exec <interpreter> [ <command> ... ]");
+    error (_("usage: interpreter-exec <interpreter> [ <command> ... ]"));
 
   old_interp = current_interpreter;
 
   interp_to_use = interp_lookup (prules[0]);
   if (interp_to_use == NULL)
-    error ("Could not find interpreter \"%s\".", prules[0]);
+    error (_("Could not find interpreter \"%s\"."), prules[0]);
 
   /* Temporarily set interpreters quiet */
   old_quiet = interp_set_quiet (old_interp, 1);
   use_quiet = interp_set_quiet (interp_to_use, 1);
 
   if (!interp_set (interp_to_use))
-    error ("Could not switch to interpreter \"%s\".", prules[0]);
+    error (_("Could not switch to interpreter \"%s\"."), prules[0]);
 
   for (i = 1; i < nrules; i++)
     {
-      if (!interp_exec (interp_to_use, prules[i]))
+      struct gdb_exception e = interp_exec (interp_to_use, prules[i]);
+      if (e.reason < 0)
        {
          interp_set (old_interp);
-         interp_set_quiet (interp_to_use, old_quiet);
-         error ("error in command: \"%s\".", prules[i]);
-         break;
+         interp_set_quiet (interp_to_use, use_quiet);
+         interp_set_quiet (old_interp, old_quiet);
+         error (_("error in command: \"%s\"."), prules[i]);
        }
     }
 
@@ -423,10 +421,11 @@ interpreter_completer (char *text, char *word)
   struct interp *interp;
 
   /* We expect only a very limited number of interpreters, so just
-     allocate room for all of them. */
+     allocate room for all of them plus one for the last that must be NULL
+     to correctly end the list. */
   for (interp = interp_list; interp != NULL; interp = interp->next)
     ++alloced;
-  matches = (char **) xmalloc (alloced * sizeof (char *));
+  matches = (char **) xcalloc (alloced + 1, sizeof (char *));
 
   num_matches = 0;
   textlen = strlen (text);
@@ -459,12 +458,6 @@ interpreter_completer (char *text, char *word)
       xfree (matches);
       matches = NULL;
     }
-  else if (num_matches < alloced)
-    {
-      matches = (char **) xrealloc ((char *) matches, ((num_matches + 1)
-                                                      * sizeof (char *)));
-      matches[num_matches] = NULL;
-    }
 
   return matches;
 }
@@ -476,9 +469,9 @@ _initialize_interpreter (void)
   struct cmd_list_element *c;
 
   c = add_cmd ("interpreter-exec", class_support,
-              interpreter_exec_cmd,
-              "Execute a command in an interpreter.  It takes two arguments:\n\
+              interpreter_exec_cmd, _("\
+Execute a command in an interpreter.  It takes two arguments:\n\
 The first argument is the name of the interpreter to use.\n\
-The second argument is the command to execute.\n", &cmdlist);
+The second argument is the command to execute.\n"), &cmdlist);
   set_cmd_completer (c, interpreter_completer);
 }
This page took 0.0269 seconds and 4 git commands to generate.