* mn10300-tdep.c (regcache.h): Delete duplicate include.
[deliverable/binutils-gdb.git] / gdb / stack.c
index 0a9052f323b5e1022aec883592a88e079903c070..c85e836801354de875c1087bdeda13d623ef4958 100644 (file)
@@ -1,8 +1,8 @@
 /* Print and select stack frames for GDB, the GNU debugger.
 
    Copyright 1986, 1987, 1988, 1989, 1990, 1991, 1992, 1993, 1994,
-   1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004 Free
-   Software Foundation, Inc.
+   1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005
+   Free Software Foundation, Inc.
 
    This file is part of GDB.
 
@@ -21,9 +21,7 @@
    Foundation, Inc., 59 Temple Place - Suite 330,
    Boston, MA 02111-1307, USA.  */
 
-#include <ctype.h>
 #include "defs.h"
-#include "gdb_string.h"
 #include "value.h"
 #include "symtab.h"
 #include "gdbtypes.h"
 #include "ui-out.h"
 #include "block.h"
 #include "stack.h"
-#include "gdb_assert.h"
 #include "dictionary.h"
+#include "exceptions.h"
 #include "reggroups.h"
 #include "regcache.h"
+#include "solib.h"
+#include "valprint.h"
 
-/* Prototypes for exported functions. */
-
-void args_info (char *, int);
-
-void locals_info (char *, int);
-
-void (*selected_frame_level_changed_hook) (int);
+#include "gdb_assert.h"
+#include <ctype.h>
+#include "gdb_string.h"
 
-void _initialize_stack (void);
+void (*deprecated_selected_frame_level_changed_hook) (int);
 
 /* Prototypes for local functions. */
 
-static void down_command (char *, int);
-
-static void down_silently_base (char *);
-
-static void down_silently_command (char *, int);
-
-static void up_command (char *, int);
-
-static void up_silently_base (char *);
-
-static void up_silently_command (char *, int);
-
-void frame_command (char *, int);
-
-static void current_frame_command (char *, int);
-
-static void print_frame_arg_vars (struct frame_info *, struct ui_file *);
-
-static void catch_info (char *, int);
-
-static void args_plus_locals_info (char *, int);
-
-static void print_frame_label_vars (struct frame_info *, int,
-                                   struct ui_file *);
-
 static void print_frame_local_vars (struct frame_info *, int,
                                    struct ui_file *);
 
-static int print_block_frame_labels (struct block *, int *,
-                                    struct ui_file *);
-
-static int print_block_frame_locals (struct block *,
-                                    struct frame_info *,
-                                    int,
-                                    struct ui_file *);
-
-static void print_frame (struct frame_info *fi, 
-                        int level, 
-                        int source, 
-                        int args, 
+static void print_frame (struct frame_info *frame, int print_level,
+                        enum print_what print_what,  int print_args,
                         struct symtab_and_line sal);
 
-static void backtrace_command (char *, int);
-
-struct frame_info *parse_frame_specification (char *);
-
-static void frame_info (char *, int);
-
-extern int addressprint;       /* Print addresses, or stay symbolic only? */
+static void set_current_sal_from_frame (struct frame_info *, int);
 
 /* Zero means do things normally; we are interacting directly with the
    user.  One means print the full filename and linenumber when a
@@ -118,65 +73,62 @@ int annotation_level = 0;
 \f
 
 struct print_stack_frame_args
-  {
-    struct frame_info *fi;
-    int level;
-    int source;
-    int args;
-  };
-
-/* Show or print the frame arguments.
-   Pass the args the way catch_errors wants them.  */
-static int print_stack_frame_stub (void *args);
+{
+  struct frame_info *frame;
+  int print_level;
+  enum print_what print_what;
+  int print_args;
+};
+
+/* Show or print the frame arguments; stub for catch_errors.  */
+
 static int
 print_stack_frame_stub (void *args)
 {
-  struct print_stack_frame_args *p = (struct print_stack_frame_args *) args;
+  struct print_stack_frame_args *p = args;
+  int center = (p->print_what == SRC_LINE || p->print_what == SRC_AND_LOC);
 
-  print_frame_info (p->fi, p->level, p->source, p->args);
+  print_frame_info (p->frame, p->print_level, p->print_what, p->print_args);
+  set_current_sal_from_frame (p->frame, center);
   return 0;
 }
 
-/* Show or print a stack frame briefly.  FRAME_INFI should be the frame info
-   and LEVEL should be its level in the stack (or -1 for level not defined).
-   This prints the level, the function executing, the arguments,
-   and the file name and line number.
-   If the pc is not at the beginning of the source line,
-   the actual pc is printed at the beginning.
-
-   If SOURCE is 1, print the source line as well.
-   If SOURCE is -1, print ONLY the source line.  */
+/* Show or print a stack frame FRAME briefly.  The output is format
+   according to PRINT_LEVEL and PRINT_WHAT printing the frame's
+   relative level, function name, argument list, and file name and
+   line number.  If the frame's PC is not at the beginning of the
+   source line, the actual PC is printed at the beginning.  */
 
 void
-print_stack_frame (struct frame_info *fi, int level, int source)
+print_stack_frame (struct frame_info *frame, int print_level,
+                  enum print_what print_what)
 {
   struct print_stack_frame_args args;
 
-  args.fi = fi;
-  args.level = level;
-  args.source = source;
-  args.args = 1;
+  args.frame = frame;
+  args.print_level = print_level;
+  args.print_what = print_what;
+  args.print_args = 1;
 
-  catch_errors (print_stack_frame_stub, (char *) &args, "", RETURN_MASK_ALL);
+  catch_errors (print_stack_frame_stub, &args, "", RETURN_MASK_ALL);
 }  
 
 struct print_args_args
 {
   struct symbol *func;
-  struct frame_info *fi;
+  struct frame_info *frame;
   struct ui_file *stream;
 };
 
-static int print_args_stub (void *);
+static int print_args_stub (void *args);
 
-/* Print nameless args on STREAM.
-   FI is the frameinfo for this frame, START is the offset
-   of the first nameless arg, and NUM is the number of nameless args to
-   print.  FIRST is nonzero if this is the first argument (not just
-   the first nameless arg).  */
+/* Print nameless arguments of frame FRAME on STREAM, where START is
+   the offset of the first nameless argument, and NUM is the number of
+   nameless arguments to print.  FIRST is nonzero if this is the first
+   argument (not just the first nameless argument).  */
 
 static void
-print_frame_nameless_args (struct frame_info *fi, long start, int num,
+print_frame_nameless_args (struct frame_info *frame, long start, int num,
                           int first, struct ui_file *stream)
 {
   int i;
@@ -186,7 +138,7 @@ print_frame_nameless_args (struct frame_info *fi, long start, int num,
   for (i = 0; i < num; i++)
     {
       QUIT;
-      argsaddr = get_frame_args_address (fi);
+      argsaddr = get_frame_args_address (frame);
       if (!argsaddr)
        return;
       arg_value = read_memory_integer (argsaddr + start, sizeof (int));
@@ -198,28 +150,24 @@ print_frame_nameless_args (struct frame_info *fi, long start, int num,
     }
 }
 
-/* Print the arguments of a stack frame, given the function FUNC
-   running in that frame (as a symbol), the info on the frame,
-   and the number of args according to the stack frame (or -1 if unknown).  */
+/* Print the arguments of frame FRAME on STREAM, given the function
+   FUNC running in that frame (as a symbol), where NUM is the number
+   of arguments according to the stack frame (or -1 if the number of
+   arguments is unknown).  */
 
-/* References here and elsewhere to "number of args according to the
-   stack frame" appear in all cases to refer to "number of ints of args
-   according to the stack frame".  At least for VAX, i386, isi.  */
+/* Note that currently the "number of argumentss according to the
+   stack frame" is only known on VAX where i refers to the "number of
+   ints of argumentss according to the stack frame".  */
 
 static void
-print_frame_args (struct symbol *func, struct frame_info *fi, int num,
-                 struct ui_file *stream)
+print_frame_args (struct symbol *func, struct frame_info *frame,
+                 int num, struct ui_file *stream)
 {
-  struct block *b = NULL;
   int first = 1;
-  struct dict_iterator iter;
-  struct symbol *sym;
-  struct value *val;
   /* Offset of next stack argument beyond the one we have seen that is
-     at the highest offset.
-     -1 if we haven't come to a stack argument yet.  */
+     at the highest offset, or -1 if we haven't come to a stack
+     argument yet.  */
   long highest_offset = -1;
-  int arg_size;
   /* Number of ints of arguments that we have printed so far.  */
   int args_printed = 0;
   struct cleanup *old_chain, *list_chain;
@@ -230,7 +178,10 @@ print_frame_args (struct symbol *func, struct frame_info *fi, int num,
 
   if (func)
     {
-      b = SYMBOL_BLOCK_VALUE (func);
+      struct block *b = SYMBOL_BLOCK_VALUE (func);
+      struct dict_iterator iter;
+      struct symbol *sym;
+      struct value *val;
 
       ALL_BLOCK_SYMBOLS (b, iter, sym)
         {
@@ -245,7 +196,7 @@ print_frame_args (struct symbol *func, struct frame_info *fi, int num,
            case LOC_REF_ARG:
              {
                long current_offset = SYMBOL_VALUE (sym);
-               arg_size = TYPE_LENGTH (SYMBOL_TYPE (sym));
+               int arg_size = TYPE_LENGTH (SYMBOL_TYPE (sym));
 
                /* Compute address of next argument by adding the size of
                   this argument and rounding to an int boundary.  */
@@ -253,17 +204,19 @@ print_frame_args (struct symbol *func, struct frame_info *fi, int num,
                  ((current_offset + arg_size + sizeof (int) - 1)
                   & ~(sizeof (int) - 1));
 
-               /* If this is the highest offset seen yet, set highest_offset.  */
+               /* If this is the highest offset seen yet, set
+                  highest_offset.  */
                if (highest_offset == -1
                    || (current_offset > highest_offset))
                  highest_offset = current_offset;
 
-               /* Add the number of ints we're about to print to args_printed.  */
+               /* Add the number of ints we're about to print to
+                  args_printed.  */
                args_printed += (arg_size + sizeof (int) - 1) / sizeof (int);
              }
 
-             /* We care about types of symbols, but don't need to keep track of
-                stack offsets in them.  */
+             /* We care about types of symbols, but don't need to
+                keep track of stack offsets in them.  */
            case LOC_REGPARM:
            case LOC_REGPARM_ADDR:
            case LOC_LOCAL_ARG:
@@ -279,48 +232,56 @@ print_frame_args (struct symbol *func, struct frame_info *fi, int num,
          /* We have to look up the symbol because arguments can have
             two entries (one a parameter, one a local) and the one we
             want is the local, which lookup_symbol will find for us.
-            This includes gcc1 (not gcc2) on the sparc when passing a
+            This includes gcc1 (not gcc2) on SPARC when passing a
             small structure and gcc2 when the argument type is float
             and it is passed as a double and converted to float by
             the prologue (in the latter case the type of the LOC_ARG
             symbol is double and the type of the LOC_LOCAL symbol is
             float).  */
-         /* But if the parameter name is null, don't try it.
-            Null parameter names occur on the RS/6000, for traceback tables.
-            FIXME, should we even print them?  */
+         /* But if the parameter name is null, don't try it.  Null
+            parameter names occur on the RS/6000, for traceback
+            tables.  FIXME, should we even print them?  */
 
          if (*DEPRECATED_SYMBOL_NAME (sym))
            {
              struct symbol *nsym;
-             nsym = lookup_symbol
-               (DEPRECATED_SYMBOL_NAME (sym),
-                b, VAR_DOMAIN, (int *) NULL, (struct symtab **) NULL);
+             nsym = lookup_symbol (DEPRECATED_SYMBOL_NAME (sym),
+                                   b, VAR_DOMAIN, NULL, NULL);
              if (SYMBOL_CLASS (nsym) == LOC_REGISTER)
                {
-                 /* There is a LOC_ARG/LOC_REGISTER pair.  This means that
-                    it was passed on the stack and loaded into a register,
-                    or passed in a register and stored in a stack slot.
-                    GDB 3.x used the LOC_ARG; GDB 4.0-4.11 used the LOC_REGISTER.
+                 /* There is a LOC_ARG/LOC_REGISTER pair.  This means
+                    that it was passed on the stack and loaded into a
+                    register, or passed in a register and stored in a
+                    stack slot.  GDB 3.x used the LOC_ARG; GDB
+                    4.0-4.11 used the LOC_REGISTER.
 
                     Reasons for using the LOC_ARG:
-                    (1) because find_saved_registers may be slow for remote
-                    debugging,
-                    (2) because registers are often re-used and stack slots
-                    rarely (never?) are.  Therefore using the stack slot is
-                    much less likely to print garbage.
+
+                    (1) Because find_saved_registers may be slow for
+                        remote debugging.
+
+                    (2) Because registers are often re-used and stack
+                        slots rarely (never?) are.  Therefore using
+                        the stack slot is much less likely to print
+                        garbage.
 
                     Reasons why we might want to use the LOC_REGISTER:
-                    (1) So that the backtrace prints the same value as
-                    "print foo".  I see no compelling reason why this needs
-                    to be the case; having the backtrace print the value which
-                    was passed in, and "print foo" print the value as modified
-                    within the called function, makes perfect sense to me.
-
-                    Additional note:  It might be nice if "info args" displayed
-                    both values.
-                    One more note:  There is a case with sparc structure passing
-                    where we need to use the LOC_REGISTER, but this is dealt with
-                    by creating a single LOC_REGPARM in symbol reading.  */
+
+                    (1) So that the backtrace prints the same value
+                        as "print foo".  I see no compelling reason
+                        why this needs to be the case; having the
+                        backtrace print the value which was passed
+                        in, and "print foo" print the value as
+                        modified within the called function, makes
+                        perfect sense to me.
+
+                    Additional note: It might be nice if "info args"
+                    displayed both values.
+
+                    One more note: There is a case with SPARC
+                    structure passing where we need to use the
+                    LOC_REGISTER, but this is dealt with by creating
+                    a single LOC_REGPARM in symbol reading.  */
 
                  /* Leave sym (the LOC_ARG) alone.  */
                  ;
@@ -338,25 +299,25 @@ print_frame_args (struct symbol *func, struct frame_info *fi, int num,
 
          list_chain = make_cleanup_ui_out_tuple_begin_end (uiout, NULL);
          fprintf_symbol_filtered (stb->stream, SYMBOL_PRINT_NAME (sym),
-                                  SYMBOL_LANGUAGE (sym), DMGL_PARAMS | DMGL_ANSI);
+                                  SYMBOL_LANGUAGE (sym),
+                                  DMGL_PARAMS | DMGL_ANSI);
          ui_out_field_stream (uiout, "name", stb);
          annotate_arg_name_end ();
          ui_out_text (uiout, "=");
 
-         /* Avoid value_print because it will deref ref parameters.  We just
-            want to print their addresses.  Print ??? for args whose address
-            we do not know.  We pass 2 as "recurse" to val_print because our
-            standard indentation here is 4 spaces, and val_print indents
-            2 for each recurse.  */
-         val = read_var_value (sym, fi);
+         /* Avoid value_print because it will deref ref parameters.
+            We just want to print their addresses.  Print ??? for
+            args whose address we do not know.  We pass 2 as
+            "recurse" to val_print because our standard indentation
+            here is 4 spaces, and val_print indents 2 for each
+            recurse.  */
+         val = read_var_value (sym, frame);
 
-         annotate_arg_value (val == NULL ? NULL : VALUE_TYPE (val));
+         annotate_arg_value (val == NULL ? NULL : value_type (val));
 
          if (val)
            {
-             val_print (VALUE_TYPE (val), VALUE_CONTENTS (val), 0,
-                        VALUE_ADDRESS (val),
-                        stb->stream, 0, 0, 2, Val_no_prettyprint);
+             common_val_print (val, stb->stream, 0, 0, 2, Val_no_prettyprint);
              ui_out_field_stream (uiout, "value", stb);
            }
          else
@@ -382,76 +343,99 @@ print_frame_args (struct symbol *func, struct frame_info *fi, int num,
       else
        start = highest_offset;
 
-      print_frame_nameless_args (fi, start, num - args_printed,
+      print_frame_nameless_args (frame, start, num - args_printed,
                                 first, stream);
     }
+
   do_cleanups (old_chain);
 }
 
-/* Pass the args the way catch_errors wants them.  */
+/* Stub for catch_errors.  */
 
 static int
 print_args_stub (void *args)
 {
+  struct print_args_args *p = args;
   int numargs;
-  struct print_args_args *p = (struct print_args_args *) args;
 
   if (FRAME_NUM_ARGS_P ())
     {
-      numargs = FRAME_NUM_ARGS (p->fi);
+      numargs = FRAME_NUM_ARGS (p->frame);
       gdb_assert (numargs >= 0);
     }
   else
     numargs = -1;
-  print_frame_args (p->func, p->fi, numargs, p->stream);
+  print_frame_args (p->func, p->frame, numargs, p->stream);
   return 0;
 }
 
-/* Print information about a frame for frame "fi" at level "level".
-   Used in "where" output, also used to emit breakpoint or step
-   messages.  
-   LEVEL is the level of the frame, or -1 if it is the
-   innermost frame but we don't want to print the level.  
-   The meaning of the SOURCE argument is: 
-   SRC_LINE: Print only source line
-   LOCATION: Print only location 
-   LOC_AND_SRC: Print location and source line.  */
+/* Set the current source and line to the location given by frame
+   FRAME, if possible.  When CENTER is true, adjust so the relevant
+   line is in the center of the next 'list'.  */
+
+static void
+set_current_sal_from_frame (struct frame_info *frame, int center)
+{
+  struct symtab_and_line sal;
+
+  find_frame_sal (frame, &sal);
+  if (sal.symtab)
+    {
+      if (center)
+        sal.line = max (sal.line - get_lines_to_list () / 2, 1);
+      set_current_source_symtab_and_line (&sal);
+    }
+}
+
+/* Print information about frame FRAME.  The output is format according
+   to PRINT_LEVEL and PRINT_WHAT and PRINT ARGS.  The meaning of
+   PRINT_WHAT is:
+   
+   SRC_LINE: Print only source line.
+   LOCATION: Print only location.
+   LOC_AND_SRC: Print location and source line.
+
+   Used in "where" output, and to emit breakpoint or step
+   messages.  */
 
 void
-print_frame_info (struct frame_info *fi, int level, int source, int args)
+print_frame_info (struct frame_info *frame, int print_level,
+                 enum print_what print_what, int print_args)
 {
   struct symtab_and_line sal;
   int source_print;
   int location_print;
 
-  if (get_frame_type (fi) == DUMMY_FRAME
-      || get_frame_type (fi) == SIGTRAMP_FRAME)
+  if (get_frame_type (frame) == DUMMY_FRAME
+      || get_frame_type (frame) == SIGTRAMP_FRAME)
     {
       struct cleanup *uiout_cleanup
        = make_cleanup_ui_out_tuple_begin_end (uiout, "frame");
 
-      annotate_frame_begin (level == -1 ? 0 : level, get_frame_pc (fi));
+      annotate_frame_begin (print_level ? frame_relative_level (frame) : 0,
+                           get_frame_pc (frame));
 
       /* Do this regardless of SOURCE because we don't have any source
          to list for this frame.  */
-      if (level >= 0)
+      if (print_level)
         {
           ui_out_text (uiout, "#");
-          ui_out_field_fmt_int (uiout, 2, ui_left, "level", level);
+          ui_out_field_fmt_int (uiout, 2, ui_left, "level",
+                               frame_relative_level (frame));
         }
       if (ui_out_is_mi_like_p (uiout))
         {
           annotate_frame_address ();
-          ui_out_field_core_addr (uiout, "addr", get_frame_pc (fi));
+          ui_out_field_core_addr (uiout, "addr", get_frame_pc (frame));
           annotate_frame_address_end ();
         }
 
-      if (get_frame_type (fi) == DUMMY_FRAME)
+      if (get_frame_type (frame) == DUMMY_FRAME)
         {
           annotate_function_call ();
           ui_out_field_string (uiout, "func", "<function called from gdb>");
        }
-      else if (get_frame_type (fi) == SIGTRAMP_FRAME)
+      else if (get_frame_type (frame) == SIGTRAMP_FRAME)
         {
          annotate_signal_handler_caller ();
           ui_out_field_string (uiout, "func", "<signal handler called>");
@@ -463,39 +447,38 @@ print_frame_info (struct frame_info *fi, int level, int source, int args)
       return;
     }
 
-  /* If fi is not the innermost frame, that normally means that fi->pc
-     points to *after* the call instruction, and we want to get the
-     line containing the call, never the next line.  But if the next
-     frame is a SIGTRAMP_FRAME or a DUMMY_FRAME, then the next frame
-     was not entered as the result of a call, and we want to get the
-     line containing fi->pc.  */
-  find_frame_sal (fi, &sal);
+  /* If FRAME is not the innermost frame, that normally means that
+     FRAME->pc points to *after* the call instruction, and we want to
+     get the line containing the call, never the next line.  But if
+     the next frame is a SIGTRAMP_FRAME or a DUMMY_FRAME, then the
+     next frame was not entered as the result of a call, and we want
+     to get the line containing FRAME->pc.  */
+  find_frame_sal (frame, &sal);
 
-  location_print = (source == LOCATION 
-                   || source == LOC_AND_ADDRESS
-                   || source == SRC_AND_LOC);
+  location_print = (print_what == LOCATION 
+                   || print_what == LOC_AND_ADDRESS
+                   || print_what == SRC_AND_LOC);
 
   if (location_print || !sal.symtab)
-    print_frame (fi, level, source, args, sal);
+    print_frame (frame, print_level, print_what, print_args, sal);
 
-  source_print = (source == SRC_LINE || source == SRC_AND_LOC);
-
-  if (sal.symtab)
-    set_current_source_symtab_and_line (&sal);
+  source_print = (print_what == SRC_LINE || print_what == SRC_AND_LOC);
 
   if (source_print && sal.symtab)
     {
-      struct symtab_and_line cursal;
       int done = 0;
-      int mid_statement = (source == SRC_LINE) && (get_frame_pc (fi) != sal.pc);
+      int mid_statement = ((print_what == SRC_LINE)
+                          && (get_frame_pc (frame) != sal.pc));
 
       if (annotation_level)
        done = identify_source_line (sal.symtab, sal.line, mid_statement,
-                                    get_frame_pc (fi));
+                                    get_frame_pc (frame));
       if (!done)
        {
-         if (print_frame_info_listing_hook)
-           print_frame_info_listing_hook (sal.symtab, sal.line, sal.line + 1, 0);
+         if (deprecated_print_frame_info_listing_hook)
+           deprecated_print_frame_info_listing_hook (sal.symtab, 
+                                                     sal.line, 
+                                                     sal.line + 1, 0);
          else
            {
              /* We used to do this earlier, but that is clearly
@@ -505,25 +488,20 @@ print_frame_info (struct frame_info *fi, int level, int source, int args)
                 when we stepi/nexti into the middle of a source
                 line. Only the command line really wants this
                 behavior. Other UIs probably would like the
-                ability to decide for themselves if it is desired. */
+                ability to decide for themselves if it is desired.  */
              if (addressprint && mid_statement)
                {
-                 ui_out_field_core_addr (uiout, "addr", get_frame_pc (fi));
+                 ui_out_field_core_addr (uiout, "addr", get_frame_pc (frame));
                  ui_out_text (uiout, "\t");
                }
 
              print_source_lines (sal.symtab, sal.line, sal.line + 1, 0);
            }
        }
-      /* Make sure we have at least a default source file */
-      set_default_source_symtab_and_line ();
-      cursal = get_current_source_symtab_and_line ();
-      cursal.line = max (sal.line - get_lines_to_list () / 2, 1);
-      set_current_source_symtab_and_line (&cursal);
     }
 
-  if (source != 0)
-    set_default_breakpoint (1, get_frame_pc (fi), sal.symtab, sal.line);
+  if (print_what != LOCATION)
+    set_default_breakpoint (1, get_frame_pc (frame), sal.symtab, sal.line);
 
   annotate_frame_end ();
 
@@ -531,54 +509,47 @@ print_frame_info (struct frame_info *fi, int level, int source, int args)
 }
 
 static void
-print_frame (struct frame_info *fi, 
-            int level, 
-            int source, 
-            int args, 
+print_frame (struct frame_info *frame, int print_level,
+            enum print_what print_what, int print_args,
             struct symtab_and_line sal)
 {
   struct symbol *func;
-  char *funname = 0;
+  char *funname = NULL;
   enum language funlang = language_unknown;
   struct ui_stream *stb;
-  struct cleanup *old_chain;
-  struct cleanup *list_chain;
+  struct cleanup *old_chain, *list_chain;
 
   stb = ui_out_stream_new (uiout);
   old_chain = make_cleanup_ui_out_stream_delete (stb);
 
-  func = find_pc_function (get_frame_address_in_block (fi));
+  func = find_pc_function (get_frame_address_in_block (frame));
   if (func)
     {
       /* In certain pathological cases, the symtabs give the wrong
          function (when we are in the first function in a file which
          is compiled without debugging symbols, the previous function
          is compiled with debugging symbols, and the "foo.o" symbol
-         that is supposed to tell us where the file with debugging symbols
-         ends has been truncated by ar because it is longer than 15
-         characters).  This also occurs if the user uses asm() to create
-         a function but not stabs for it (in a file compiled -g).
+         that is supposed to tell us where the file with debugging
+         symbols ends has been truncated by ar because it is longer
+         than 15 characters).  This also occurs if the user uses asm()
+         to create a function but not stabs for it (in a file compiled
+         with -g).
 
          So look in the minimal symbol tables as well, and if it comes
          up with a larger address for the function use that instead.
-         I don't think this can ever cause any problems; there shouldn't
-         be any minimal symbols in the middle of a function; if this is
-         ever changed many parts of GDB will need to be changed (and we'll
-         create a find_pc_minimal_function or some such).  */
+         I don't think this can ever cause any problems; there
+         shouldn't be any minimal symbols in the middle of a function;
+         if this is ever changed many parts of GDB will need to be
+         changed (and we'll create a find_pc_minimal_function or some
+         such).  */
+
+      struct minimal_symbol *msymbol =
+       lookup_minimal_symbol_by_pc (get_frame_address_in_block (frame));
 
-      struct minimal_symbol *msymbol = lookup_minimal_symbol_by_pc (get_frame_address_in_block (fi));
       if (msymbol != NULL
          && (SYMBOL_VALUE_ADDRESS (msymbol)
              > BLOCK_START (SYMBOL_BLOCK_VALUE (func))))
        {
-#if 0
-         /* There is no particular reason to think the line number
-            information is wrong.  Someone might have just put in
-            a label with asm() but left the line numbers alone.  */
-         /* In this case we have no way of knowing the source file
-            and line number, so don't print them.  */
-         sal.symtab = 0;
-#endif
          /* We also don't know anything about the function besides
             its address and name.  */
          func = 0;
@@ -587,34 +558,37 @@ print_frame (struct frame_info *fi,
        }
       else
        {
-         /* I'd like to use SYMBOL_PRINT_NAME() here, to display the
-            demangled name that we already have stored in the symbol
-            table, but we stored a version with DMGL_PARAMS turned
-            on, and here we don't want to display parameters. So call
-            the demangler again, with DMGL_ANSI only. (Yes, I know
-            that printf_symbol_filtered() will again try to demangle
-            the name on the fly, but the issue is that if
-            cplus_demangle() fails here, it'll fail there too. So we
-            want to catch the failure ("demangled==NULL" case below)
-            here, while we still have our hands on the function
-            symbol.) */
-         char *demangled;
          funname = DEPRECATED_SYMBOL_NAME (func);
          funlang = SYMBOL_LANGUAGE (func);
          if (funlang == language_cplus)
            {
-             demangled = cplus_demangle (funname, DMGL_ANSI);
+             /* It seems appropriate to use SYMBOL_PRINT_NAME() here,
+                to display the demangled name that we already have
+                stored in the symbol table, but we stored a version
+                with DMGL_PARAMS turned on, and here we don't want to
+                display parameters. So call the demangler again, with
+                DMGL_ANSI only.
+
+                Yes, printf_symbol_filtered() will again try to
+                demangle the name on the fly, but the issue is that
+                if cplus_demangle() fails here, it will fail there
+                too. So we want to catch the failure (where DEMANGLED
+                is NULL below) here, while we still have our hands on
+                the function symbol.)  */
+             char *demangled = cplus_demangle (funname, DMGL_ANSI);
              if (demangled == NULL)
                /* If the demangler fails, try the demangled name from
-                  the symbol table. This'll have parameters, but
-                  that's preferable to diplaying a mangled name. */
+                  the symbol table. That'll have parameters, but
+                  that's preferable to displaying a mangled name.  */
                funname = SYMBOL_PRINT_NAME (func);
            }
        }
     }
   else
     {
-      struct minimal_symbol *msymbol = lookup_minimal_symbol_by_pc (get_frame_address_in_block (fi));
+      struct minimal_symbol *msymbol = 
+       lookup_minimal_symbol_by_pc (get_frame_address_in_block (frame));
+
       if (msymbol != NULL)
        {
          funname = DEPRECATED_SYMBOL_NAME (msymbol);
@@ -622,44 +596,45 @@ print_frame (struct frame_info *fi,
        }
     }
 
-  annotate_frame_begin (level == -1 ? 0 : level, get_frame_pc (fi));
+  annotate_frame_begin (print_level ? frame_relative_level (frame) : 0,
+                       get_frame_pc (frame));
 
   list_chain = make_cleanup_ui_out_tuple_begin_end (uiout, "frame");
 
-  if (level >= 0)
+  if (print_level)
     {
       ui_out_text (uiout, "#");
-      ui_out_field_fmt_int (uiout, 2, ui_left, "level", level);
+      ui_out_field_fmt_int (uiout, 2, ui_left, "level",
+                           frame_relative_level (frame));
     }
   if (addressprint)
-    if (get_frame_pc (fi) != sal.pc
-       || !sal.symtab
-       || source == LOC_AND_ADDRESS)
+    if (get_frame_pc (frame) != sal.pc || !sal.symtab
+       || print_what == LOC_AND_ADDRESS)
       {
        annotate_frame_address ();
-       ui_out_field_core_addr (uiout, "addr", get_frame_pc (fi));
+       ui_out_field_core_addr (uiout, "addr", get_frame_pc (frame));
        annotate_frame_address_end ();
        ui_out_text (uiout, " in ");
       }
   annotate_frame_function_name ();
-  fprintf_symbol_filtered (stb->stream, funname ? funname : "??", funlang,
-                          DMGL_ANSI);
+  fprintf_symbol_filtered (stb->stream, funname ? funname : "??",
+                          funlang, DMGL_ANSI);
   ui_out_field_stream (uiout, "func", stb);
   ui_out_wrap_hint (uiout, "   ");
   annotate_frame_args ();
       
   ui_out_text (uiout, " (");
-  if (args)
+  if (print_args)
     {
       struct print_args_args args;
       struct cleanup *args_list_chain;
-      args.fi = fi;
+      args.frame = frame;
       args.func = func;
       args.stream = gdb_stdout;
       args_list_chain = make_cleanup_ui_out_list_begin_end (uiout, "args");
       catch_errors (print_args_stub, &args, "", RETURN_MASK_ALL);
-      /* FIXME: args must be a list. If one argument is a string it will
-                have " that will not be properly escaped.  */
+      /* FIXME: ARGS must be a list. If one argument is a string it
+         will have " that will not be properly escaped.  */
       /* Invoke ui_out_tuple_end.  */
       do_cleanups (args_list_chain);
       QUIT;
@@ -672,6 +647,12 @@ print_frame (struct frame_info *fi,
       ui_out_text (uiout, " at ");
       annotate_frame_source_file ();
       ui_out_field_string (uiout, "file", sal.symtab->filename);
+      if (ui_out_is_mi_like_p (uiout))
+       {
+         const char *fullname = symtab_to_fullname (sal.symtab);
+         if (fullname != NULL)
+           ui_out_field_string (uiout, "fullname", fullname);
+       }
       annotate_frame_source_file_end ();
       ui_out_text (uiout, ":");
       annotate_frame_source_line ();
@@ -679,10 +660,13 @@ print_frame (struct frame_info *fi,
       annotate_frame_source_end ();
     }
 
-#ifdef PC_SOLIB
   if (!funname || (!sal.symtab || !sal.symtab->filename))
     {
-      char *lib = PC_SOLIB (get_frame_pc (fi));
+#ifdef PC_SOLIB
+      char *lib = PC_SOLIB (get_frame_pc (frame));
+#else
+      char *lib = solib_address (get_frame_pc (frame));
+#endif
       if (lib)
        {
          annotate_frame_where ();
@@ -691,7 +675,6 @@ print_frame (struct frame_info *fi,
          ui_out_field_string (uiout, "from", lib);
        }
     }
-#endif /* PC_SOLIB */
 
   /* do_cleanups will call ui_out_tuple_end() for us.  */
   do_cleanups (list_chain);
@@ -699,135 +682,145 @@ print_frame (struct frame_info *fi,
   do_cleanups (old_chain);
 }
 \f
-/* Show the frame info.  If this is the tui, it will be shown in 
-   the source display otherwise, nothing is done */
+/* Show the frame info.  If this is the tui, it will be shown in the
+   source display otherwise, nothing is done.  */
+
 void
-show_stack_frame (struct frame_info *fi)
+show_stack_frame (struct frame_info *frame)
 {
 }
 \f
 
-/* Read a frame specification in whatever the appropriate format is.
-   Call error() if the specification is in any way invalid (i.e.
-   this function never returns NULL).  */
+/* Read a frame specification in whatever the appropriate format is
+   from FRAME_EXP.  Call error(), printing MESSAGE, if the
+   specification is in any way invalid (so this function never returns
+   NULL).  When SEPECTED_P is non-NULL set its target to indicate that
+   the default selected frame was used.  */
 
-struct frame_info *
-parse_frame_specification (char *frame_exp)
+static struct frame_info *
+parse_frame_specification_1 (const char *frame_exp, const char *message,
+                            int *selected_frame_p)
 {
-  int numargs = 0;
-#define        MAXARGS 4
-  CORE_ADDR args[MAXARGS];
-  int level;
+  int numargs;
+  struct value *args[4];
+  CORE_ADDR addrs[ARRAY_SIZE (args)];
 
-  if (frame_exp)
+  if (frame_exp == NULL)
+    numargs = 0;
+  else
     {
-      char *addr_string, *p;
+      char *addr_string;
       struct cleanup *tmp_cleanup;
 
-      while (*frame_exp == ' ')
-       frame_exp++;
-
-      while (*frame_exp)
+      numargs = 0;
+      while (1)
        {
-         if (numargs > MAXARGS)
-           error ("Too many args in frame specification");
-         /* Parse an argument.  */
-         for (p = frame_exp; *p && *p != ' '; p++)
-           ;
-         addr_string = savestring (frame_exp, p - frame_exp);
-
-         {
-           struct value *vp;
-
-           tmp_cleanup = make_cleanup (xfree, addr_string);
-
-           /* NOTE: we call parse_and_eval and then both
-              value_as_long and value_as_address rather than calling
-              parse_and_eval_long and parse_and_eval_address because
-              of the issue of potential side effects from evaluating
-              the expression.  */
-           vp = parse_and_eval (addr_string);
-           if (numargs == 0)
-             level = value_as_long (vp);
-
-           args[numargs++] = value_as_address (vp);
-           do_cleanups (tmp_cleanup);
-         }
+         char *addr_string;
+         struct cleanup *cleanup;
+         const char *p;
+
+         /* Skip leading white space, bail of EOL.  */
+         while (isspace (*frame_exp))
+           frame_exp++;
+         if (!*frame_exp)
+           break;
 
-         /* Skip spaces, move to possible next arg.  */
-         while (*p == ' ')
-           p++;
+         /* Parse the argument, extract it, save it.  */
+         for (p = frame_exp;
+              *p && !isspace (*p);
+              p++);
+         addr_string = savestring (frame_exp, p - frame_exp);
          frame_exp = p;
+         cleanup = make_cleanup (xfree, addr_string);
+         
+         /* NOTE: Parse and evaluate expression, but do not use
+            functions such as parse_and_eval_long or
+            parse_and_eval_address to also extract the value.
+            Instead value_as_long and value_as_address are used.
+            This avoids problems with expressions that contain
+            side-effects.  */
+         if (numargs >= ARRAY_SIZE (args))
+           error (_("Too many args in frame specification"));
+         args[numargs++] = parse_and_eval (addr_string);
+
+         do_cleanups (cleanup);
        }
     }
 
-  switch (numargs)
+  /* If no args, default to the selected frame.  */
+  if (numargs == 0)
     {
-    case 0:
-      if (deprecated_selected_frame == NULL)
-       error ("No selected frame.");
-      return deprecated_selected_frame;
-      /* NOTREACHED */
-    case 1:
-      {
-       struct frame_info *fid =
-       find_relative_frame (get_current_frame (), &level);
-       struct frame_info *tfid;
-
-       if (level == 0)
-         /* find_relative_frame was successful */
-         return fid;
-
-       /* If SETUP_ARBITRARY_FRAME is defined, then frame specifications
-          take at least 2 addresses.  It is important to detect this case
-          here so that "frame 100" does not give a confusing error message
-          like "frame specification requires two addresses".  This of course
-          does not solve the "frame 100" problem for machines on which
-          a frame specification can be made with one address.  To solve
-          that, we need a new syntax for a specifying a frame by address.
-          I think the cleanest syntax is $frame(0x45) ($frame(0x23,0x45) for
-          two args, etc.), but people might think that is too much typing,
-          so I guess *0x23,0x45 would be a possible alternative (commas
-          really should be used instead of spaces to delimit; using spaces
-          normally works in an expression).  */
-#ifdef SETUP_ARBITRARY_FRAME
-       error ("No frame %s", paddr_d (args[0]));
-#endif
+      if (selected_frame_p != NULL)
+       (*selected_frame_p) = 1;
+      return get_selected_frame (message);
+    }
 
-       /* If (s)he specifies the frame with an address, he deserves what
-          (s)he gets.  Still, give the highest one that matches.  */
+  /* None of the remaining use the selected frame.  */
+  if (selected_frame_p != NULL)
+    (*selected_frame_p) = 0;
 
-       for (fid = get_current_frame ();
-            fid && get_frame_base (fid) != args[0];
-            fid = get_prev_frame (fid))
-         ;
+  /* Assume the single arg[0] is an integer, and try using that to
+     select a frame relative to current.  */
+  if (numargs == 1)
+    {
+      struct frame_info *fid;
+      int level = value_as_long (args[0]);
+      fid = find_relative_frame (get_current_frame (), &level);
+      if (level == 0)
+       /* find_relative_frame was successful */
+       return fid;
+    }
 
-       if (fid)
-         while ((tfid = get_prev_frame (fid)) &&
-                (get_frame_base (tfid) == args[0]))
-           fid = tfid;
+  /* Convert each value into a corresponding address.  */
+  {
+    int i;
+    for (i = 0; i < numargs; i++)
+      addrs[i] = value_as_address (args[0]);
+  }
 
-       /* We couldn't identify the frame as an existing frame, but
-          perhaps we can create one with a single argument.  */
+  /* Assume that the single arg[0] is an address, use that to identify
+     a frame with a matching ID.  Should this also accept stack/pc or
+     stack/pc/special.  */
+  if (numargs == 1)
+    {
+      struct frame_id id = frame_id_build_wild (addrs[0]);
+      struct frame_info *fid;
+
+      /* If (s)he specifies the frame with an address, he deserves
+        what (s)he gets.  Still, give the highest one that matches.
+        (NOTE: cagney/2004-10-29: Why highest, or outer-most, I don't
+        know).  */
+      for (fid = get_current_frame ();
+          fid != NULL;
+          fid = get_prev_frame (fid))
+       {
+         if (frame_id_eq (id, get_frame_id (fid)))
+           {
+             while (frame_id_eq (id, frame_unwind_id (fid)))
+               fid = get_prev_frame (fid);
+             return fid;
+           }
+       }
       }
 
-    default:
-#ifdef SETUP_ARBITRARY_FRAME
-      return SETUP_ARBITRARY_FRAME (numargs, args);
-#else
-      /* Usual case.  Do it here rather than have everyone supply
-         a SETUP_ARBITRARY_FRAME that does this.  */
-      if (numargs == 1)
-       return create_new_frame (args[0], 0);
-      error ("Too many args in frame specification");
-#endif
-      /* NOTREACHED */
-    }
-  /* NOTREACHED */
+  /* We couldn't identify the frame as an existing frame, but
+     perhaps we can create one with a single argument.  */
+  if (numargs == 1)
+    return create_new_frame (addrs[0], 0);
+  else if (numargs == 2)
+    return create_new_frame (addrs[0], addrs[1]);
+  else
+    error (_("Too many args in frame specification"));
 }
 
-/* Print verbosely the selected frame or the frame at address ADDR.
-   This means absolutely all information in the frame is printed.  */
+static struct frame_info *
+parse_frame_specification (char *frame_exp)
+{
+  return parse_frame_specification_1 (frame_exp, NULL, NULL);
+}
+
+/* Print verbosely the selected frame or the frame at address
+   ADDR_EXP.  Absolutely all information in the frame is printed.  */
 
 static void
 frame_info (char *addr_exp, int from_tty)
@@ -841,9 +834,9 @@ frame_info (char *addr_exp, int from_tty)
   char *funname = 0;
   enum language funlang = language_unknown;
   const char *pc_regname;
+  int selected_frame_p;
 
-  if (!target_has_stack)
-    error ("No stack.");
+  fi = parse_frame_specification_1 (addr_exp, "No stack.", &selected_frame_p);
 
   /* Name of the value returned by get_frame_pc().  Per comments, "pc"
      is not a good name.  */
@@ -859,47 +852,41 @@ frame_info (char *addr_exp, int from_tty)
        get_frame_pc().  */
     pc_regname = "pc";
 
-  fi = parse_frame_specification (addr_exp);
-  if (fi == NULL)
-    error ("Invalid frame specified.");
-
   find_frame_sal (fi, &sal);
   func = get_frame_function (fi);
   /* FIXME: cagney/2002-11-28: Why bother?  Won't sal.symtab contain
-     the same value.  */
+     the same value?  */
   s = find_pc_symtab (get_frame_pc (fi));
   if (func)
     {
-      /* I'd like to use SYMBOL_PRINT_NAME() here, to display
-       * the demangled name that we already have stored in
-       * the symbol table, but we stored a version with
-       * DMGL_PARAMS turned on, and here we don't want
-       * to display parameters. So call the demangler again,
-       * with DMGL_ANSI only. RT
-       * (Yes, I know that printf_symbol_filtered() will
-       * again try to demangle the name on the fly, but
-       * the issue is that if cplus_demangle() fails here,
-       * it'll fail there too. So we want to catch the failure
-       * ("demangled==NULL" case below) here, while we still
-       * have our hands on the function symbol.)
-       */
-      char *demangled;
+      /* It seems appropriate to use SYMBOL_PRINT_NAME() here, to
+        display the demangled name that we already have stored in the
+        symbol table, but we stored a version with DMGL_PARAMS turned
+        on, and here we don't want to display parameters. So call the
+        demangler again, with DMGL_ANSI only.
+
+        Yes, printf_symbol_filtered() will again try to demangle the
+        name on the fly, but the issue is that if cplus_demangle()
+        fails here, it will fail there too. So we want to catch the
+        failure (where DEMANGLED is NULL below) here, while we still
+        have our hands on the function symbol.)  */
       funname = DEPRECATED_SYMBOL_NAME (func);
       funlang = SYMBOL_LANGUAGE (func);
       if (funlang == language_cplus)
        {
-         demangled = cplus_demangle (funname, DMGL_ANSI);
-         /* If the demangler fails, try the demangled name
-          * from the symbol table. This'll have parameters,
-          * but that's preferable to diplaying a mangled name.
-          */
+         char *demangled = cplus_demangle (funname, DMGL_ANSI);
+         /* If the demangler fails, try the demangled name from the
+            symbol table. That'll have parameters, but that's
+            preferable to displaying a mangled name.  */
          if (demangled == NULL)
            funname = SYMBOL_PRINT_NAME (func);
        }
     }
   else
     {
-      struct minimal_symbol *msymbol = lookup_minimal_symbol_by_pc (get_frame_pc (fi));
+      struct minimal_symbol *msymbol;
+
+      msymbol = lookup_minimal_symbol_by_pc (get_frame_pc (fi));
       if (msymbol != NULL)
        {
          funname = DEPRECATED_SYMBOL_NAME (msymbol);
@@ -908,21 +895,19 @@ frame_info (char *addr_exp, int from_tty)
     }
   calling_frame_info = get_prev_frame (fi);
 
-  if (!addr_exp && frame_relative_level (deprecated_selected_frame) >= 0)
+  if (selected_frame_p && frame_relative_level (fi) >= 0)
     {
-      printf_filtered ("Stack level %d, frame at ",
-                      frame_relative_level (deprecated_selected_frame));
-      print_address_numeric (get_frame_base (fi), 1, gdb_stdout);
-      printf_filtered (":\n");
+      printf_filtered (_("Stack level %d, frame at "),
+                      frame_relative_level (fi));
     }
   else
     {
-      printf_filtered ("Stack frame at ");
-      print_address_numeric (get_frame_base (fi), 1, gdb_stdout);
-      printf_filtered (":\n");
+      printf_filtered (_("Stack frame at "));
     }
+  deprecated_print_address_numeric (get_frame_base (fi), 1, gdb_stdout);
+  printf_filtered (":\n");
   printf_filtered (" %s = ", pc_regname);
-  print_address_numeric (get_frame_pc (fi), 1, gdb_stdout);
+  deprecated_print_address_numeric (get_frame_pc (fi), 1, gdb_stdout);
 
   wrap_here ("   ");
   if (funname)
@@ -937,20 +922,13 @@ frame_info (char *addr_exp, int from_tty)
   puts_filtered ("; ");
   wrap_here ("    ");
   printf_filtered ("saved %s ", pc_regname);
-  print_address_numeric (frame_pc_unwind (fi), 1, gdb_stdout);
+  deprecated_print_address_numeric (frame_pc_unwind (fi), 1, gdb_stdout);
   printf_filtered ("\n");
 
-  {
-    int frameless;
-    frameless = FRAMELESS_FUNCTION_INVOCATION (fi);
-    if (frameless)
-      printf_filtered (" (FRAMELESS),");
-  }
-
   if (calling_frame_info)
     {
       printf_filtered (" called by frame at ");
-      print_address_numeric (get_frame_base (calling_frame_info),
+      deprecated_print_address_numeric (get_frame_base (calling_frame_info),
                             1, gdb_stdout);
     }
   if (get_next_frame (fi) && calling_frame_info)
@@ -959,7 +937,7 @@ frame_info (char *addr_exp, int from_tty)
   if (get_next_frame (fi))
     {
       printf_filtered (" caller of frame at ");
-      print_address_numeric (get_frame_base (get_next_frame (fi)), 1,
+      deprecated_print_address_numeric (get_frame_base (get_next_frame (fi)), 1,
                             gdb_stdout);
     }
   if (get_next_frame (fi) || calling_frame_info)
@@ -979,7 +957,7 @@ frame_info (char *addr_exp, int from_tty)
     else
       {
        printf_filtered (" Arglist at ");
-       print_address_numeric (arg_list, 1, gdb_stdout);
+       deprecated_print_address_numeric (arg_list, 1, gdb_stdout);
        printf_filtered (",");
 
        if (!FRAME_NUM_ARGS_P ())
@@ -1011,14 +989,11 @@ frame_info (char *addr_exp, int from_tty)
     else
       {
        printf_filtered (" Locals at ");
-       print_address_numeric (arg_list, 1, gdb_stdout);
+       deprecated_print_address_numeric (arg_list, 1, gdb_stdout);
        printf_filtered (",");
       }
   }
 
-  if (DEPRECATED_FRAME_INIT_SAVED_REGS_P ()
-      && deprecated_get_frame_saved_regs (fi) == NULL)
-    DEPRECATED_FRAME_INIT_SAVED_REGS (fi);
   /* Print as much information as possible on the location of all the
      registers.  */
   {
@@ -1043,23 +1018,23 @@ frame_info (char *addr_exp, int from_tty)
                               &realnum, NULL);
        if (!optimized && lval == not_lval)
          {
-           char value[MAX_REGISTER_SIZE];
+           gdb_byte value[MAX_REGISTER_SIZE];
            CORE_ADDR sp;
            frame_register_unwind (fi, SP_REGNUM, &optimized, &lval, &addr,
                                   &realnum, value);
            /* NOTE: cagney/2003-05-22: This is assuming that the
                stack pointer was packed as an unsigned integer.  That
                may or may not be valid.  */
-           sp = extract_unsigned_integer (value, DEPRECATED_REGISTER_RAW_SIZE (SP_REGNUM));
+           sp = extract_unsigned_integer (value, register_size (current_gdbarch, SP_REGNUM));
            printf_filtered (" Previous frame's sp is ");
-           print_address_numeric (sp, 1, gdb_stdout);
+           deprecated_print_address_numeric (sp, 1, gdb_stdout);
            printf_filtered ("\n");
            need_nl = 0;
          }
        else if (!optimized && lval == lval_memory)
          {
            printf_filtered (" Previous frame's sp at ");
-           print_address_numeric (addr, 1, gdb_stdout);
+           deprecated_print_address_numeric (addr, 1, gdb_stdout);
            printf_filtered ("\n");
            need_nl = 0;
          }
@@ -1092,7 +1067,7 @@ frame_info (char *addr_exp, int from_tty)
                puts_filtered (",");
              wrap_here (" ");
              printf_filtered (" %s at ", REGISTER_NAME (i));
-             print_address_numeric (addr, 1, gdb_stdout);
+             deprecated_print_address_numeric (addr, 1, gdb_stdout);
              count++;
            }
        }
@@ -1101,37 +1076,9 @@ frame_info (char *addr_exp, int from_tty)
   }
 }
 
-#if 0
-/* Set a limit on the number of frames printed by default in a
-   backtrace.  */
-
-static int backtrace_limit;
+/* Print briefly all stack frames or just the innermost COUNT_EXP
+   frames.  */
 
-static void
-set_backtrace_limit_command (char *count_exp, int from_tty)
-{
-  int count = parse_and_eval_long (count_exp);
-
-  if (count < 0)
-    error ("Negative argument not meaningful as backtrace limit.");
-
-  backtrace_limit = count;
-}
-
-static void
-backtrace_limit_info (char *arg, int from_tty)
-{
-  if (arg)
-    error ("\"Info backtrace-limit\" takes no arguments.");
-
-  printf_unfiltered ("Backtrace limit: %d.\n", backtrace_limit);
-}
-#endif
-
-/* Print briefly all stack frames or just the innermost COUNT frames.  */
-
-static void backtrace_command_1 (char *count_exp, int show_locals,
-                                int from_tty);
 static void
 backtrace_command_1 (char *count_exp, int show_locals, int from_tty)
 {
@@ -1142,10 +1089,10 @@ backtrace_command_1 (char *count_exp, int show_locals, int from_tty)
   int trailing_level;
 
   if (!target_has_stack)
-    error ("No stack.");
+    error (_("No stack."));
 
-  /* The following code must do two things.  First, it must
-     set the variable TRAILING to the frame from which we should start
+  /* The following code must do two things.  First, it must set the
+     variable TRAILING to the frame from which we should start
      printing.  Second, it must set the variable count to the number
      of frames which we should print, or -1 if all of them.  */
   trailing = get_current_frame ();
@@ -1153,7 +1100,7 @@ backtrace_command_1 (char *count_exp, int show_locals, int from_tty)
   /* The target can be in a state where there is no valid frames
      (e.g., just connected). */
   if (trailing == NULL)
-    error ("No stack.");
+    error (_("No stack."));
 
   trailing_level = 0;
   if (count_exp)
@@ -1172,8 +1119,8 @@ backtrace_command_1 (char *count_exp, int show_locals, int from_tty)
              current = get_prev_frame (current);
            }
 
-         /* Will stop when CURRENT reaches the top of the stack.  TRAILING
-            will be COUNT below it.  */
+         /* Will stop when CURRENT reaches the top of the stack.
+            TRAILING will be COUNT below it.  */
          while (current)
            {
              QUIT;
@@ -1192,26 +1139,22 @@ backtrace_command_1 (char *count_exp, int show_locals, int from_tty)
     {
       struct partial_symtab *ps;
 
-      /* Read in symbols for all of the frames.  Need to do this in
-         separate pass so that "Reading in symbols for xxx" messages
-         don't screw up the appearance of the backtrace.  Also
-         if people have strong opinions against reading symbols for
+      /* Read in symbols for all of the frames.  Need to do this in a
+         separate pass so that "Reading in symbols for xxx" messages
+         don't screw up the appearance of the backtrace.  Also if
+         people have strong opinions against reading symbols for
          backtrace this may have to be an option.  */
       i = count;
-      for (fi = trailing;
-          fi != NULL && i--;
-          fi = get_prev_frame (fi))
+      for (fi = trailing; fi != NULL && i--; fi = get_prev_frame (fi))
        {
          QUIT;
          ps = find_pc_psymtab (get_frame_address_in_block (fi));
          if (ps)
-           PSYMTAB_TO_SYMTAB (ps);     /* Force syms to come in */
+           PSYMTAB_TO_SYMTAB (ps); /* Force syms to come in.  */
        }
     }
 
-  for (i = 0, fi = trailing;
-       fi && count--;
-       i++, fi = get_prev_frame (fi))
+  for (i = 0, fi = trailing; fi && count--; i++, fi = get_prev_frame (fi))
     {
       QUIT;
 
@@ -1219,100 +1162,118 @@ backtrace_command_1 (char *count_exp, int show_locals, int from_tty)
          means further attempts to backtrace would fail (on the other
          hand, perhaps the code does or could be fixed to make sure
          the frame->prev field gets set to NULL in that case).  */
-      print_frame_info (fi, trailing_level + i, 0, 1);
+      print_frame_info (fi, 1, LOCATION, 1);
       if (show_locals)
        print_frame_local_vars (fi, 1, gdb_stdout);
     }
 
   /* If we've stopped before the end, mention that.  */
   if (fi && from_tty)
-    printf_filtered ("(More stack frames follow...)\n");
+    printf_filtered (_("(More stack frames follow...)\n"));
+}
+
+struct backtrace_command_args
+{
+  char *count_exp;
+  int show_locals;
+  int from_tty;
+};
+
+/* Stub for catch_errors.  */
+
+static int
+backtrace_command_stub (void *data)
+{
+  struct backtrace_command_args *args = data;
+  backtrace_command_1 (args->count_exp, args->show_locals, args->from_tty);
+  return 0;
 }
 
 static void
 backtrace_command (char *arg, int from_tty)
 {
-  struct cleanup *old_chain = (struct cleanup *) NULL;
-  char **argv = (char **) NULL;
-  int argIndicatingFullTrace = (-1), totArgLen = 0, argc = 0;
-  char *argPtr = arg;
+  struct cleanup *old_chain = NULL;
+  int fulltrace_arg = -1, arglen = 0, argc = 0;
+  struct backtrace_command_args btargs;
 
-  if (arg != (char *) NULL)
+  if (arg)
     {
+      char **argv;
       int i;
 
       argv = buildargv (arg);
       old_chain = make_cleanup_freeargv (argv);
       argc = 0;
-      for (i = 0; (argv[i] != (char *) NULL); i++)
+      for (i = 0; argv[i]; i++)
        {
          unsigned int j;
 
-         for (j = 0; (j < strlen (argv[i])); j++)
+         for (j = 0; j < strlen (argv[i]); j++)
            argv[i][j] = tolower (argv[i][j]);
 
-         if (argIndicatingFullTrace < 0 && subset_compare (argv[i], "full"))
-           argIndicatingFullTrace = argc;
+         if (fulltrace_arg < 0 && subset_compare (argv[i], "full"))
+           fulltrace_arg = argc;
          else
            {
+             arglen += strlen (argv[i]);
              argc++;
-             totArgLen += strlen (argv[i]);
            }
        }
-      totArgLen += argc;
-      if (argIndicatingFullTrace >= 0)
+      arglen += argc;
+      if (fulltrace_arg >= 0)
        {
-         if (totArgLen > 0)
+         if (arglen > 0)
            {
-             argPtr = (char *) xmalloc (totArgLen + 1);
-             if (!argPtr)
-               nomem (0);
-             else
+             arg = xmalloc (arglen + 1);
+             memset (arg, 0, arglen + 1);
+             for (i = 0; i < (argc + 1); i++)
                {
-                 memset (argPtr, 0, totArgLen + 1);
-                 for (i = 0; (i < (argc + 1)); i++)
+                 if (i != fulltrace_arg)
                    {
-                     if (i != argIndicatingFullTrace)
-                       {
-                         strcat (argPtr, argv[i]);
-                         strcat (argPtr, " ");
-                       }
+                     strcat (arg, argv[i]);
+                     strcat (arg, " ");
                    }
                }
            }
          else
-           argPtr = (char *) NULL;
+           arg = NULL;
        }
     }
 
-  backtrace_command_1 (argPtr, (argIndicatingFullTrace >= 0), from_tty);
+  btargs.count_exp = arg;
+  btargs.show_locals = (fulltrace_arg >= 0);
+  btargs.from_tty = from_tty;
+  catch_errors (backtrace_command_stub, &btargs, "", RETURN_MASK_ERROR);
 
-  if (argIndicatingFullTrace >= 0 && totArgLen > 0)
-    xfree (argPtr);
+  if (fulltrace_arg >= 0 && arglen > 0)
+    xfree (arg);
 
   if (old_chain)
     do_cleanups (old_chain);
 }
 
-static void backtrace_full_command (char *arg, int from_tty);
 static void
 backtrace_full_command (char *arg, int from_tty)
 {
-  backtrace_command_1 (arg, 1, from_tty);
+  struct backtrace_command_args btargs;
+  btargs.count_exp = arg;
+  btargs.show_locals = 1;
+  btargs.from_tty = from_tty;
+  catch_errors (backtrace_command_stub, &btargs, "", RETURN_MASK_ERROR);
 }
 \f
 
-/* Print the local variables of a block B active in FRAME.
+/* Print the local variables of a block B active in FRAME on STREAM.
    Return 1 if any variables were printed; 0 otherwise.  */
 
 static int
-print_block_frame_locals (struct block *b, struct frame_info *fi,
+print_block_frame_locals (struct block *b, struct frame_info *frame,
                          int num_tabs, struct ui_file *stream)
 {
   struct dict_iterator iter;
-  int j;
   struct symbol *sym;
   int values_printed = 0;
+  int j;
 
   ALL_BLOCK_SYMBOLS (b, iter, sym)
     {
@@ -1328,7 +1289,7 @@ print_block_frame_locals (struct block *b, struct frame_info *fi,
            fputs_filtered ("\t", stream);
          fputs_filtered (SYMBOL_PRINT_NAME (sym), stream);
          fputs_filtered (" = ", stream);
-         print_variable_value (sym, fi, stream);
+         print_variable_value (sym, frame, stream);
          fprintf_filtered (stream, "\n");
          break;
 
@@ -1337,6 +1298,7 @@ print_block_frame_locals (struct block *b, struct frame_info *fi,
          break;
        }
     }
+
   return values_printed;
 }
 
@@ -1352,7 +1314,7 @@ print_block_frame_labels (struct block *b, int *have_default,
 
   ALL_BLOCK_SYMBOLS (b, iter, sym)
     {
-      if (DEPRECATED_STREQ (DEPRECATED_SYMBOL_NAME (sym), "default"))
+      if (strcmp (DEPRECATED_SYMBOL_NAME (sym), "default") == 0)
        {
          if (*have_default)
            continue;
@@ -1367,28 +1329,27 @@ print_block_frame_labels (struct block *b, int *have_default,
          if (addressprint)
            {
              fprintf_filtered (stream, " ");
-             print_address_numeric (SYMBOL_VALUE_ADDRESS (sym), 1, stream);
+             deprecated_print_address_numeric (SYMBOL_VALUE_ADDRESS (sym), 1, stream);
            }
          fprintf_filtered (stream, " in file %s, line %d\n",
                            sal.symtab->filename, sal.line);
        }
     }
+
   return values_printed;
 }
 
-/* Print on STREAM all the local variables in frame FRAME,
-   including all the blocks active in that frame
-   at its current pc.
+/* Print on STREAM all the local variables in frame FRAME, including
+   all the blocks active in that frame at its current PC.
 
-   Returns 1 if the job was done,
-   or 0 if nothing was printed because we have no info
-   on the function running in FRAME.  */
+   Returns 1 if the job was done, or 0 if nothing was printed because
+   we have no info on the function running in FRAME.  */
 
 static void
-print_frame_local_vars (struct frame_info *fi, int num_tabs,
+print_frame_local_vars (struct frame_info *frame, int num_tabs,
                        struct ui_file *stream)
 {
-  struct block *block = get_frame_block (fi, 0);
+  struct block *block = get_frame_block (frame, 0);
   int values_printed = 0;
 
   if (block == 0)
@@ -1397,36 +1358,33 @@ print_frame_local_vars (struct frame_info *fi, int num_tabs,
       return;
     }
 
-  while (block != 0)
+  while (block)
     {
-      if (print_block_frame_locals (block, fi, num_tabs, stream))
+      if (print_block_frame_locals (block, frame, num_tabs, stream))
        values_printed = 1;
-      /* After handling the function's top-level block, stop.
-         Don't continue to its superblock, the block of
-         per-file symbols.  */
+      /* After handling the function's top-level block, stop.  Don't
+         continue to its superblock, the block of per-file symbols.  */
       if (BLOCK_FUNCTION (block))
        break;
       block = BLOCK_SUPERBLOCK (block);
     }
 
   if (!values_printed)
-    {
-      fprintf_filtered (stream, "No locals.\n");
-    }
+    fprintf_filtered (stream, _("No locals.\n"));
 }
 
 /* Same, but print labels.  */
 
 static void
-print_frame_label_vars (struct frame_info *fi, int this_level_only,
+print_frame_label_vars (struct frame_info *frame, int this_level_only,
                        struct ui_file *stream)
 {
   struct blockvector *bl;
-  struct block *block = get_frame_block (fi, 0);
+  struct block *block = get_frame_block (frame, 0);
   int values_printed = 0;
   int index, have_default = 0;
   char *blocks_printed;
-  CORE_ADDR pc = get_frame_pc (fi);
+  CORE_ADDR pc = get_frame_pc (frame);
 
   if (block == 0)
     {
@@ -1435,7 +1393,7 @@ print_frame_label_vars (struct frame_info *fi, int this_level_only,
     }
 
   bl = blockvector_for_pc (BLOCK_END (block) - 4, &index);
-  blocks_printed = (char *) alloca (BLOCKVECTOR_NBLOCKS (bl) * sizeof (char));
+  blocks_printed = alloca (BLOCKVECTOR_NBLOCKS (bl) * sizeof (char));
   memset (blocks_printed, 0, BLOCKVECTOR_NBLOCKS (bl) * sizeof (char));
 
   while (block != 0)
@@ -1444,9 +1402,9 @@ print_frame_label_vars (struct frame_info *fi, int this_level_only,
       int last_index;
 
       if (bl != blockvector_for_pc (end, &index))
-       error ("blockvector blotch");
+       error (_("blockvector blotch"));
       if (BLOCKVECTOR_BLOCK (bl, index) != block)
-       error ("blockvector botch");
+       error (_("blockvector botch"));
       last_index = BLOCKVECTOR_NBLOCKS (bl);
       index += 1;
 
@@ -1460,7 +1418,8 @@ print_frame_label_vars (struct frame_info *fi, int this_level_only,
        {
          if (blocks_printed[index] == 0)
            {
-             if (print_block_frame_labels (BLOCKVECTOR_BLOCK (bl, index), &have_default, stream))
+             if (print_block_frame_labels (BLOCKVECTOR_BLOCK (bl, index),
+                                           &have_default, stream))
                values_printed = 1;
              blocks_printed[index] = 1;
            }
@@ -1471,26 +1430,22 @@ print_frame_label_vars (struct frame_info *fi, int this_level_only,
       if (values_printed && this_level_only)
        return;
 
-      /* After handling the function's top-level block, stop.
-         Don't continue to its superblock, the block of
-         per-file symbols.  */
+      /* After handling the function's top-level block, stop.  Don't
+         continue to its superblock, the block of per-file symbols.  */
       if (BLOCK_FUNCTION (block))
        break;
       block = BLOCK_SUPERBLOCK (block);
     }
 
   if (!values_printed && !this_level_only)
-    {
-      fprintf_filtered (stream, "No catches.\n");
-    }
+    fprintf_filtered (stream, _("No catches.\n"));
 }
 
 void
 locals_info (char *args, int from_tty)
 {
-  if (!deprecated_selected_frame)
-    error ("No frame selected.");
-  print_frame_local_vars (deprecated_selected_frame, 0, gdb_stdout);
+  print_frame_local_vars (get_selected_frame (_("No frame selected.")),
+                         0, gdb_stdout);
 }
 
 static void
@@ -1502,30 +1457,24 @@ catch_info (char *ignore, int from_tty)
   sal = target_enable_exception_callback (EX_EVENT_CATCH, 1);
   if (sal)
     {
-      /* Currently not handling this */
-      /* Ideally, here we should interact with the C++ runtime
-         system to find the list of active handlers, etc. */
-      fprintf_filtered (gdb_stdout, "Info catch not supported with this target/compiler combination.\n");
-#if 0
-      if (!deprecated_selected_frame)
-       error ("No frame selected.");
-#endif
+      /* Currently not handling this.  Ideally, here we should
+         interact with the C++ runtime system to find the list of
+         active handlers, etc.  */
+      fprintf_filtered (gdb_stdout, _("\
+Info catch not supported with this target/compiler combination.\n"));
     }
   else
     {
-      /* Assume g++ compiled code -- old v 4.16 behaviour */
-      if (!deprecated_selected_frame)
-       error ("No frame selected.");
-
-      print_frame_label_vars (deprecated_selected_frame, 0, gdb_stdout);
+      /* Assume g++ compiled code; old GDB 4.16 behaviour.  */
+      print_frame_label_vars (get_selected_frame (_("No frame selected.")),
+                             0, gdb_stdout);
     }
 }
 
 static void
-print_frame_arg_vars (struct frame_info *fi,
-                     struct ui_file *stream)
+print_frame_arg_vars (struct frame_info *frame, struct ui_file *stream)
 {
-  struct symbol *func = get_frame_function (fi);
+  struct symbol *func = get_frame_function (frame);
   struct block *b;
   struct dict_iterator iter;
   struct symbol *sym, *sym2;
@@ -1533,7 +1482,7 @@ print_frame_arg_vars (struct frame_info *fi,
 
   if (func == 0)
     {
-      fprintf_filtered (stream, "No symbol table info available.\n");
+      fprintf_filtered (stream, _("No symbol table info available.\n"));
       return;
     }
 
@@ -1565,8 +1514,8 @@ print_frame_arg_vars (struct frame_info *fi,
             are not combined in symbol-reading.  */
 
          sym2 = lookup_symbol (DEPRECATED_SYMBOL_NAME (sym),
-                  b, VAR_DOMAIN, (int *) NULL, (struct symtab **) NULL);
-         print_variable_value (sym2, fi, stream);
+                               b, VAR_DOMAIN, NULL, NULL);
+         print_variable_value (sym2, frame, stream);
          fprintf_filtered (stream, "\n");
          break;
 
@@ -1575,18 +1524,16 @@ print_frame_arg_vars (struct frame_info *fi,
          break;
        }
     }
+
   if (!values_printed)
-    {
-      fprintf_filtered (stream, "No arguments.\n");
-    }
+    fprintf_filtered (stream, _("No arguments.\n"));
 }
 
 void
 args_info (char *ignore, int from_tty)
 {
-  if (!deprecated_selected_frame)
-    error ("No frame selected.");
-  print_frame_arg_vars (deprecated_selected_frame, gdb_stdout);
+  print_frame_arg_vars (get_selected_frame (_("No frame selected.")),
+                       gdb_stdout);
 }
 
 
@@ -1598,16 +1545,14 @@ args_plus_locals_info (char *ignore, int from_tty)
 }
 \f
 
-/* Select frame FI.  Also print the stack frame and show the source if
-   this is the tui version.  */
+/* Select frame FRAME.  Also print the stack frame and show the source
+   if this is the tui version.  */
 static void
-select_and_print_frame (struct frame_info *fi)
+select_and_print_frame (struct frame_info *frame)
 {
-  select_frame (fi);
-  if (fi)
-    {
-      print_stack_frame (fi, frame_relative_level (fi), 1);
-    }
+  select_frame (frame);
+  if (frame)
+    print_stack_frame (frame, 1, SRC_AND_LOC);
 }
 \f
 /* Return the symbol-block in which the selected frame is executing.
@@ -1624,7 +1569,7 @@ get_selected_block (CORE_ADDR *addr_in_block)
     return 0;
 
   /* NOTE: cagney/2002-11-28: Why go to all this effort to not create
-     a selected/current frame?  Perhaphs this function is called,
+     a selected/current frame?  Perhaps this function is called,
      indirectly, by WFI in "infrun.c" where avoiding the creation of
      an inner most frame is very important (it slows down single
      step).  I suspect, though that this was true in the deep dark
@@ -1653,102 +1598,80 @@ get_selected_block (CORE_ADDR *addr_in_block)
    how much farther the original request asked to go.  */
 
 struct frame_info *
-find_relative_frame (struct frame_info *frame,
-                    int *level_offset_ptr)
+find_relative_frame (struct frame_info *frame, int *level_offset_ptr)
 {
-  struct frame_info *prev;
-  struct frame_info *frame1;
-
-  /* Going up is simple: just do get_prev_frame enough times
-     or until initial frame is reached.  */
+  /* Going up is simple: just call get_prev_frame enough times or
+     until the initial frame is reached.  */
   while (*level_offset_ptr > 0)
     {
-      prev = get_prev_frame (frame);
-      if (prev == 0)
+      struct frame_info *prev = get_prev_frame (frame);
+      if (!prev)
        break;
       (*level_offset_ptr)--;
       frame = prev;
     }
+
   /* Going down is just as simple.  */
-  if (*level_offset_ptr < 0)
+  while (*level_offset_ptr < 0)
     {
-      while (*level_offset_ptr < 0)
-       {
-         frame1 = get_next_frame (frame);
-         if (!frame1)
-           break;
-         frame = frame1;
-         (*level_offset_ptr)++;
-       }
+      struct frame_info *next = get_next_frame (frame);
+      if (!next)
+       break;
+      (*level_offset_ptr)++;
+      frame = next;
     }
+
   return frame;
 }
 
-/* The "select_frame" command.  With no arg, NOP.
-   With arg LEVEL_EXP, select the frame at level LEVEL if it is a
-   valid level.  Otherwise, treat level_exp as an address expression
-   and select it.  See parse_frame_specification for more info on proper
-   frame expressions. */
+/* The "select_frame" command.  With no argument this is a NOP.
+   Select the frame at level LEVEL_EXP if it is a valid level.
+   Otherwise, treat LEVEL_EXP as an address expression and select it.
+
+   See parse_frame_specification for more info on proper frame
+   expressions.  */
 
 void
 select_frame_command (char *level_exp, int from_tty)
 {
-  struct frame_info *frame;
-  int level = frame_relative_level (deprecated_selected_frame);
-
-  if (!target_has_stack)
-    error ("No stack.");
-
-  frame = parse_frame_specification (level_exp);
-
-  select_frame (frame);
-  if (level != frame_relative_level (deprecated_selected_frame))
-    selected_frame_level_changed_event (frame_relative_level (deprecated_selected_frame));
+  select_frame (parse_frame_specification_1 (level_exp, "No stack.", NULL));
 }
 
-/* The "frame" command.  With no arg, print selected frame briefly.
-   With arg, behaves like select_frame and then prints the selected
-   frame.  */
+/* The "frame" command.  With no argument, print the selected frame
+   briefly.  With an argument, behave like select_frame and then print
+   the selected frame.  */
 
-void
+static void
 frame_command (char *level_exp, int from_tty)
 {
   select_frame_command (level_exp, from_tty);
-  print_stack_frame (deprecated_selected_frame,
-                    frame_relative_level (deprecated_selected_frame), 1);
+  print_stack_frame (get_selected_frame (NULL), 1, SRC_AND_LOC);
 }
 
-/* The XDB Compatibility command to print the current frame. */
+/* The XDB Compatibility command to print the current frame.  */
 
 static void
 current_frame_command (char *level_exp, int from_tty)
 {
-  if (target_has_stack == 0 || deprecated_selected_frame == 0)
-    error ("No stack.");
-  print_stack_frame (deprecated_selected_frame,
-                         frame_relative_level (deprecated_selected_frame), 1);
+  print_stack_frame (get_selected_frame (_("No stack.")), 1, SRC_AND_LOC);
 }
 
-/* Select the frame up one or COUNT stack levels
-   from the previously selected frame, and print it briefly.  */
+/* Select the frame up one or COUNT_EXP stack levels from the
+   previously selected frame, and print it briefly.  */
 
 static void
 up_silently_base (char *count_exp)
 {
-  struct frame_info *fi;
-  int count = 1, count1;
+  struct frame_info *frame;
+  int count = 1;
+
   if (count_exp)
     count = parse_and_eval_long (count_exp);
-  count1 = count;
 
-  if (target_has_stack == 0 || deprecated_selected_frame == 0)
-    error ("No stack.");
-
-  fi = find_relative_frame (deprecated_selected_frame, &count1);
-  if (count1 != 0 && count_exp == 0)
-    error ("Initial frame selected; you cannot go up.");
-  select_frame (fi);
-  selected_frame_level_changed_event (frame_relative_level (deprecated_selected_frame));
+  frame = find_relative_frame (get_selected_frame ("No stack."), &count);
+  if (count != 0 && count_exp == NULL)
+    error (_("Initial frame selected; you cannot go up."));
+  select_frame (frame);
 }
 
 static void
@@ -1761,39 +1684,32 @@ static void
 up_command (char *count_exp, int from_tty)
 {
   up_silently_base (count_exp);
-  print_stack_frame (deprecated_selected_frame,
-                    frame_relative_level (deprecated_selected_frame), 1);
+  print_stack_frame (get_selected_frame (NULL), 1, SRC_AND_LOC);
 }
 
-/* Select the frame down one or COUNT stack levels
-   from the previously selected frame, and print it briefly.  */
+/* Select the frame down one or COUNT_EXP stack levels from the previously
+   selected frame, and print it briefly.  */
 
 static void
 down_silently_base (char *count_exp)
 {
   struct frame_info *frame;
-  int count = -1, count1;
+  int count = -1;
   if (count_exp)
     count = -parse_and_eval_long (count_exp);
-  count1 = count;
-
-  if (target_has_stack == 0 || deprecated_selected_frame == 0)
-    error ("No stack.");
 
-  frame = find_relative_frame (deprecated_selected_frame, &count1);
-  if (count1 != 0 && count_exp == 0)
+  frame = find_relative_frame (get_selected_frame ("No stack."), &count);
+  if (count != 0 && count_exp == NULL)
     {
+      /* We only do this if COUNT_EXP is not specified.  That way
+         "down" means to really go down (and let me know if that is
+         impossible), but "down 9999" can be used to mean go all the
+         way down without getting an error.  */
 
-      /* We only do this if count_exp is not specified.  That way "down"
-         means to really go down (and let me know if that is
-         impossible), but "down 9999" can be used to mean go all the way
-         down without getting an error.  */
-
-      error ("Bottom (i.e., innermost) frame selected; you cannot go down.");
+      error (_("Bottom (innermost) frame selected; you cannot go down."));
     }
 
   select_frame (frame);
-  selected_frame_level_changed_event (frame_relative_level (deprecated_selected_frame));
 }
 
 static void
@@ -1806,10 +1722,10 @@ static void
 down_command (char *count_exp, int from_tty)
 {
   down_silently_base (count_exp);
-  print_stack_frame (deprecated_selected_frame,
-                    frame_relative_level (deprecated_selected_frame), 1);
+  print_stack_frame (get_selected_frame (NULL), 1, SRC_AND_LOC);
 }
 \f
+
 void
 return_command (char *retval_exp, int from_tty)
 {
@@ -1817,13 +1733,7 @@ return_command (char *retval_exp, int from_tty)
   struct value *return_value = NULL;
   const char *query_prefix = "";
 
-  /* FIXME: cagney/2003-10-20: Perform a minimal existance test on the
-     target.  If that fails, error out.  For the moment don't rely on
-     get_selected_frame as it's error message is the the singularly
-     obscure "No registers".  */
-  if (!target_has_registers)
-    error ("No selected frame.");
-  thisfun = get_frame_function (get_selected_frame ());
+  thisfun = get_frame_function (get_selected_frame ("No selected frame."));
 
   /* Compute the return value.  If the computation triggers an error,
      let it bail.  If the return type can't be handled, set
@@ -1843,11 +1753,12 @@ return_command (char *retval_exp, int from_tty)
        return_type = TYPE_TARGET_TYPE (SYMBOL_TYPE (thisfun));
       if (return_type == NULL)
        return_type = builtin_type_int;
+      CHECK_TYPEDEF (return_type);
       return_value = value_cast (return_type, return_value);
 
       /* Make sure the value is fully evaluated.  It may live in the
          stack frame we're about to pop.  */
-      if (VALUE_LAZY (return_value))
+      if (value_lazy (return_value))
        value_fetch_lazy (return_value);
 
       if (TYPE_CODE (return_type) == TYPE_CODE_VOID)
@@ -1855,7 +1766,7 @@ return_command (char *retval_exp, int from_tty)
            return-value's location.  However, do still evaluate the
            return expression so that, even when the expression result
            is discarded, side effects such as "return i++" still
-           occure.  */
+           occur.  */
        return_value = NULL;
       /* FIXME: cagney/2004-01-17: If the architecture implements both
          return_value and extract_returned_value_address, should allow
@@ -1889,13 +1800,13 @@ If you continue, the return value that you specified will be ignored.\n";
     {
       int confirmed;
       if (thisfun == NULL)
-       confirmed = query ("%sMake selected stack frame return now? ",
+       confirmed = query (_("%sMake selected stack frame return now? "),
                           query_prefix);
       else
-       confirmed = query ("%sMake %s return now? ", query_prefix,
+       confirmed = query (_("%sMake %s return now? "), query_prefix,
                           SYMBOL_PRINT_NAME (thisfun));
       if (!confirmed)
-       error ("Not confirmed");
+       error (_("Not confirmed"));
     }
 
   /* NOTE: cagney/2003-01-18: Is this silly?  Rather than pop each
@@ -1905,13 +1816,13 @@ If you continue, the return value that you specified will be ignored.\n";
   /* First discard all frames inner-to the selected frame (making the
      selected frame current).  */
   {
-    struct frame_id selected_id = get_frame_id (get_selected_frame ());
+    struct frame_id selected_id = get_frame_id (get_selected_frame (NULL));
     while (!frame_id_eq (selected_id, get_frame_id (get_current_frame ())))
       {
        if (frame_id_inner (selected_id, get_frame_id (get_current_frame ())))
          /* Caught in the safety net, oops!  We've gone way past the
              selected frame.  */
-         error ("Problem while popping stack frames (corrupt stack?)");
+         error (_("Problem while popping stack frames (corrupt stack?)"));
        frame_pop (get_current_frame ());
       }
   }
@@ -1923,43 +1834,19 @@ If you continue, the return value that you specified will be ignored.\n";
   /* Store RETURN_VAUE in the just-returned register set.  */
   if (return_value != NULL)
     {
-      struct type *return_type = VALUE_TYPE (return_value);
-      if (!gdbarch_return_value_p (current_gdbarch))
-       {
-         STORE_RETURN_VALUE (return_type, current_regcache,
-                             VALUE_CONTENTS (return_value));
-       }
-      /* FIXME: cagney/2004-01-17: If extract_returned_value_address
-         is available and the function is using
-         RETURN_VALUE_STRUCT_CONVENTION, should use it to find the
-         address of the returned value so that it can be assigned.  */
-      else
-       {
-         gdb_assert (gdbarch_return_value (current_gdbarch, return_type,
-                                           NULL, NULL, NULL)
-                     == RETURN_VALUE_REGISTER_CONVENTION);
-         gdbarch_return_value (current_gdbarch, return_type,
-                               current_regcache, NULL /*read*/,
-                               VALUE_CONTENTS (return_value) /*write*/);
-       }
+      struct type *return_type = value_type (return_value);
+      gdb_assert (gdbarch_return_value (current_gdbarch, return_type,
+                                       NULL, NULL, NULL)
+                 == RETURN_VALUE_REGISTER_CONVENTION);
+      gdbarch_return_value (current_gdbarch, return_type,
+                           current_regcache, NULL /*read*/,
+                           value_contents (return_value) /*write*/);
     }
 
   /* If we are at the end of a call dummy now, pop the dummy frame
      too.  */
-  /* NOTE: cagney/2003-01-18: Is this silly?  Instead of popping all
-     the frames in sequence, should this code just pop the dummy frame
-     directly?  */
-#ifdef DEPRECATED_CALL_DUMMY_HAS_COMPLETED
-  /* Since all up-to-date architectures return direct to the dummy
-     breakpoint address, a dummy frame has, by definition, always
-     completed.  Hence this method is no longer needed.  */
-  if (DEPRECATED_CALL_DUMMY_HAS_COMPLETED (read_pc(), read_sp (),
-                                          get_frame_base (get_current_frame ())))
-    frame_pop (get_current_frame ());
-#else
   if (get_frame_type (get_current_frame ()) == DUMMY_FRAME)
     frame_pop (get_current_frame ());
-#endif
 
   /* If interactive, print the frame that is now current.  */
   if (from_tty)
@@ -1968,54 +1855,51 @@ If you continue, the return value that you specified will be ignored.\n";
     select_frame_command ("0", 0);
 }
 
-/* Sets the scope to input function name, provided that the
-   function is within the current stack frame */
+/* Sets the scope to input function name, provided that the function
+   is within the current stack frame */
 
 struct function_bounds
 {
   CORE_ADDR low, high;
 };
 
-static void func_command (char *arg, int from_tty);
 static void
 func_command (char *arg, int from_tty)
 {
-  struct frame_info *fp;
+  struct frame_info *frame;
   int found = 0;
   struct symtabs_and_lines sals;
   int i;
   int level = 1;
-  struct function_bounds *func_bounds = (struct function_bounds *) NULL;
+  struct function_bounds *func_bounds = NULL;
 
-  if (arg != (char *) NULL)
+  if (arg != NULL)
     return;
 
-  fp = parse_frame_specification ("0");
+  frame = parse_frame_specification ("0");
   sals = decode_line_spec (arg, 1);
   func_bounds = (struct function_bounds *) xmalloc (
                              sizeof (struct function_bounds) * sals.nelts);
   for (i = 0; (i < sals.nelts && !found); i++)
     {
-      if (sals.sals[i].pc == (CORE_ADDR) 0 ||
-         find_pc_partial_function (sals.sals[i].pc,
-                                   (char **) NULL,
-                                   &func_bounds[i].low,
-                                   &func_bounds[i].high) == 0)
+      if (sals.sals[i].pc == 0
+         || find_pc_partial_function (sals.sals[i].pc, NULL,
+                                      &func_bounds[i].low,
+                                      &func_bounds[i].high) == 0)
        {
-         func_bounds[i].low =
-           func_bounds[i].high = (CORE_ADDR) NULL;
+         func_bounds[i].low = func_bounds[i].high = 0;
        }
     }
 
   do
     {
       for (i = 0; (i < sals.nelts && !found); i++)
-       found = (get_frame_pc (fp) >= func_bounds[i].low &&
-                get_frame_pc (fp) < func_bounds[i].high);
+       found = (get_frame_pc (frame) >= func_bounds[i].low
+                && get_frame_pc (frame) < func_bounds[i].high);
       if (!found)
        {
          level = 1;
-         fp = find_relative_frame (fp, &level);
+         frame = find_relative_frame (frame, &level);
        }
     }
   while (!found && level == 0);
@@ -2024,9 +1908,9 @@ func_command (char *arg, int from_tty)
     xfree (func_bounds);
 
   if (!found)
-    printf_filtered ("'%s' not within current stack frame.\n", arg);
-  else if (fp != deprecated_selected_frame)
-    select_and_print_frame (fp);
+    printf_filtered (_("'%s' not within current stack frame.\n"), arg);
+  else if (frame != deprecated_selected_frame)
+    select_and_print_frame (frame);
 }
 
 /* Gets the language of the current frame.  */
@@ -2034,30 +1918,32 @@ func_command (char *arg, int from_tty)
 enum language
 get_frame_language (void)
 {
-  struct symtab *s;
-  enum language flang;         /* The language of the current frame */
+  struct frame_info *frame = deprecated_selected_frame;
 
-  if (deprecated_selected_frame)
+  if (frame)
     {
       /* We determine the current frame language by looking up its
-         associated symtab.  To retrieve this symtab, we use the frame PC.
-         However we cannot use the frame pc as is, because it usually points
-         to the instruction following the "call", which is sometimes the first
-         instruction of another function.  So we rely on
-         get_frame_address_in_block(), it provides us with a PC which is
-         guaranteed to be inside the frame's code block.  */
-      s = find_pc_symtab (get_frame_address_in_block (deprecated_selected_frame));
+         associated symtab.  To retrieve this symtab, we use the frame
+         PC.  However we cannot use the frame PC as is, because it
+         usually points to the instruction following the "call", which
+         is sometimes the first instruction of another function.  So
+         we rely on get_frame_address_in_block(), it provides us with
+         a PC that is guaranteed to be inside the frame's code
+         block.  */
+      CORE_ADDR pc = get_frame_address_in_block (frame);
+      struct symtab *s = find_pc_symtab (pc);
+
       if (s)
-       flang = s->language;
-      else
-       flang = language_unknown;
+       return s->language;
     }
-  else
-    flang = language_unknown;
 
-  return flang;
+  return language_unknown;
 }
 \f
+
+/* Provide a prototype to silence -Wmissing-prototypes.  */
+void _initialize_stack (void);
+
 void
 _initialize_stack (void)
 {
@@ -2065,91 +1951,92 @@ _initialize_stack (void)
   backtrace_limit = 30;
 #endif
 
-  add_com ("return", class_stack, return_command,
-          "Make selected stack frame return to its caller.\n\
+  add_com ("return", class_stack, return_command, _("\
+Make selected stack frame return to its caller.\n\
 Control remains in the debugger, but when you continue\n\
 execution will resume in the frame above the one now selected.\n\
-If an argument is given, it is an expression for the value to return.");
-
-  add_com ("up", class_stack, up_command,
-          "Select and print stack frame that called this one.\n\
-An argument says how many frames up to go.");
-  add_com ("up-silently", class_support, up_silently_command,
-          "Same as the `up' command, but does not print anything.\n\
-This is useful in command scripts.");
-
-  add_com ("down", class_stack, down_command,
-          "Select and print stack frame called by this one.\n\
-An argument says how many frames down to go.");
+If an argument is given, it is an expression for the value to return."));
+
+  add_com ("up", class_stack, up_command, _("\
+Select and print stack frame that called this one.\n\
+An argument says how many frames up to go."));
+  add_com ("up-silently", class_support, up_silently_command, _("\
+Same as the `up' command, but does not print anything.\n\
+This is useful in command scripts."));
+
+  add_com ("down", class_stack, down_command, _("\
+Select and print stack frame called by this one.\n\
+An argument says how many frames down to go."));
   add_com_alias ("do", "down", class_stack, 1);
   add_com_alias ("dow", "down", class_stack, 1);
-  add_com ("down-silently", class_support, down_silently_command,
-          "Same as the `down' command, but does not print anything.\n\
-This is useful in command scripts.");
+  add_com ("down-silently", class_support, down_silently_command, _("\
+Same as the `down' command, but does not print anything.\n\
+This is useful in command scripts."));
 
-  add_com ("frame", class_stack, frame_command,
-          "Select and print a stack frame.\n\
+  add_com ("frame", class_stack, frame_command, _("\
+Select and print a stack frame.\n\
 With no argument, print the selected stack frame.  (See also \"info frame\").\n\
 An argument specifies the frame to select.\n\
 It can be a stack frame number or the address of the frame.\n\
 With argument, nothing is printed if input is coming from\n\
-a command file or a user-defined command.");
+a command file or a user-defined command."));
 
   add_com_alias ("f", "frame", class_stack, 1);
 
   if (xdb_commands)
     {
       add_com ("L", class_stack, current_frame_command,
-              "Print the current stack frame.\n");
+              _("Print the current stack frame.\n"));
       add_com_alias ("V", "frame", class_stack, 1);
     }
-  add_com ("select-frame", class_stack, select_frame_command,
-          "Select a stack frame without printing anything.\n\
+  add_com ("select-frame", class_stack, select_frame_command, _("\
+Select a stack frame without printing anything.\n\
 An argument specifies the frame to select.\n\
-It can be a stack frame number or the address of the frame.\n");
+It can be a stack frame number or the address of the frame.\n"));
 
-  add_com ("backtrace", class_stack, backtrace_command,
-          "Print backtrace of all stack frames, or innermost COUNT frames.\n\
+  add_com ("backtrace", class_stack, backtrace_command, _("\
+Print backtrace of all stack frames, or innermost COUNT frames.\n\
 With a negative argument, print outermost -COUNT frames.\n\
-Use of the 'full' qualifier also prints the values of the local variables.\n");
+Use of the 'full' qualifier also prints the values of the local variables.\n"));
   add_com_alias ("bt", "backtrace", class_stack, 0);
   if (xdb_commands)
     {
       add_com_alias ("t", "backtrace", class_stack, 0);
-      add_com ("T", class_stack, backtrace_full_command,
-              "Print backtrace of all stack frames, or innermost COUNT frames \n\
+      add_com ("T", class_stack, backtrace_full_command, _("\
+Print backtrace of all stack frames, or innermost COUNT frames \n\
 and the values of the local variables.\n\
 With a negative argument, print outermost -COUNT frames.\n\
-Usage: T <count>\n");
+Usage: T <count>\n"));
     }
 
   add_com_alias ("where", "backtrace", class_alias, 0);
   add_info ("stack", backtrace_command,
-           "Backtrace of the stack, or innermost COUNT frames.");
+           _("Backtrace of the stack, or innermost COUNT frames."));
   add_info_alias ("s", "stack", 1);
   add_info ("frame", frame_info,
-           "All about selected stack frame, or frame at ADDR.");
+           _("All about selected stack frame, or frame at ADDR."));
   add_info_alias ("f", "frame", 1);
   add_info ("locals", locals_info,
-           "Local variables of current stack frame.");
+           _("Local variables of current stack frame."));
   add_info ("args", args_info,
-           "Argument variables of current stack frame.");
+           _("Argument variables of current stack frame."));
   if (xdb_commands)
     add_com ("l", class_info, args_plus_locals_info,
-            "Argument and local variables of current stack frame.");
+            _("Argument and local variables of current stack frame."));
 
   if (dbx_commands)
-    add_com ("func", class_stack, func_command,
-      "Select the stack frame that contains <func>.\nUsage: func <name>\n");
+    add_com ("func", class_stack, func_command, _("\
+Select the stack frame that contains <func>.\n\
+Usage: func <name>\n"));
 
   add_info ("catch", catch_info,
-           "Exceptions that can be caught in the current stack frame.");
+           _("Exceptions that can be caught in the current stack frame."));
 
 #if 0
-  add_cmd ("backtrace-limit", class_stack, set_backtrace_limit_command,
-  "Specify maximum number of frames for \"backtrace\" to print by default.",
+  add_cmd ("backtrace-limit", class_stack, set_backtrace_limit_command, _(\
+"Specify maximum number of frames for \"backtrace\" to print by default."),
           &setlist);
-  add_info ("backtrace-limit", backtrace_limit_info,
-     "The maximum number of frames for \"backtrace\" to print by default.");
+  add_info ("backtrace-limit", backtrace_limit_info, _("\
+The maximum number of frames for \"backtrace\" to print by default."));
 #endif
 }
This page took 0.067296 seconds and 4 git commands to generate.