diff options
author | Joel Brobecker <brobecker@gnat.com> | 2012-11-12 21:28:51 +0000 |
---|---|---|
committer | Joel Brobecker <brobecker@gnat.com> | 2012-11-12 21:28:51 +0000 |
commit | b2c7d45a4c44572a7584b46c96b6bb21057bfeea (patch) | |
tree | 5d1d4b78f4fe1cc83da76c51c7b786e9b6a59836 | |
parent | 31594462cabd1148f94aea7c2064ba0564568316 (diff) | |
download | gdb-b2c7d45a4c44572a7584b46c96b6bb21057bfeea.zip gdb-b2c7d45a4c44572a7584b46c96b6bb21057bfeea.tar.gz gdb-b2c7d45a4c44572a7584b46c96b6bb21057bfeea.tar.bz2 |
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.
-rw-r--r-- | gdb/ChangeLog | 6 | ||||
-rw-r--r-- | gdb/frame.c | 3 | ||||
-rw-r--r-- | gdb/frame.h | 12 |
3 files changed, 8 insertions, 13 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index bd1216e..510c0e1 100644 --- 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 71f0ba4..52498ba 100644 --- 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 fa80663..c0559d9 100644 --- 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); |