Document replacement for frame_register_read (deprecated).

gdb/ChangeLog:

        * frame.h (frame_register_read): Remove FIXME comment.
        * frame.c (frame_register_read): Add suggestion explaining
        which function to use in place of this one.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index bd1216ee5391b10d5bf1dbbf2f732f08678768ae..510c0e1e85c8e1746792775b971103a8b8ce798a 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2012-11-12  Joel Brobecker  <brobecker@adacore.com>
+
+       * frame.h (frame_register_read): Remove FIXME comment.
+       * frame.c (frame_register_read): Add suggestion explaining
+       which function to use in place of this one.
+
 2012-11-12  Tom Tromey  <tromey@redhat.com>
 
        * python/python.c (start_type_printers): Initialize 'result_obj'.

diff --git a/gdb/frame.c b/gdb/frame.c
index 71f0ba44bbbe2e5e96f39c929b9f015a35b96422..52498bad6cf4c57cd1e95dde69be6e5919473e2f 100644 (file)
--- a/gdb/frame.c
+++ b/gdb/frame.c
@@ -1088,7 +1088,8 @@ put_frame_register (struct frame_info *frame, int regnum,
     }
 }
 
-/* frame_register_read ()
+/* This function is deprecated.  Use get_frame_register_value instead,
+   which provides more accurate information.
 
    Find and return the value of REGNUM for the specified stack frame.
    The number of bytes copied is REGISTER_SIZE (REGNUM).

diff --git a/gdb/frame.h b/gdb/frame.h
index fa80663bd7f21eb757520e94c50d97df3f5269c0..c0559d90dcb50d7078f8e5fa97f6331234a46e22 100644 (file)
--- a/gdb/frame.h
+++ b/gdb/frame.h
@@ -673,18 +673,6 @@ extern struct frame_info *block_innermost_frame (const struct block *);
 
 extern int deprecated_pc_in_call_dummy (struct gdbarch *gdbarch, CORE_ADDR pc);
 
-/* FIXME: cagney/2003-02-02: Should be deprecated or replaced with a
-   function called get_frame_register_p().  This slightly weird (and
-   older) variant of get_frame_register() returns zero (indicating the
-   register value is unavailable/invalid) if either: the register
-   isn't cached; or the register has been optimized out; or the
-   register contents are unavailable (because they haven't been
-   collected in a traceframe).  Problem is, neither check is exactly
-   correct.  A register can't be optimized out (it may not have been
-   saved as part of a function call); The fact that a register isn't
-   in the register cache doesn't mean that the register isn't
-   available (it could have been fetched from memory).  */
-
 extern int frame_register_read (struct frame_info *frame, int regnum,
                                gdb_byte *buf);