From: Pedro Alves Date: Fri, 29 Nov 2013 15:25:46 +0000 (+0000) Subject: UNWIND_NULL_ID is no longer used anywhere. Update comments. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=20e1ca3bc11b03d85dcee5a155fdf5290476028b;p=binutils-gdb.git UNWIND_NULL_ID is no longer used anywhere. Update comments. Unfortunately, UNWIND_NULL_ID is exported to Python as gdb.FRAME_UNWIND_NULL_ID so we can't really eliminate it. (I'd assume scripts just check the result of Frame.unwind_stop_reason, and compare it to gdb.FRAME_UNWIND_NO_REASON. That at most, they'll pass the result of Frame.unwind_stop_reason to gdb.frame_stop_reason_string. I'd prefer to just get rid of it, but because we make an API promise, we get to keep this around for compatibility, in case a script does refer to gdb.FRAME_UNWIND_NULL_ID directly.) gdb/ 2013-11-29 Pedro Alves * unwind_stop_reasons.def (UNWIND_NULL_ID): Update comment. gdb/doc/ 2013-11-29 Pedro Alves * gdb.texinfo (Frames In Python) : Update comment. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 5f0626c6521..3e02284c6d2 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2013-11-29 Pedro Alves + + * unwind_stop_reasons.def (UNWIND_NULL_ID): Update comment. + 2013-11-29 Pedro Alves * breakpoint.c (build_target_condition_list): Release previous diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog index 141d47f4ee9..87d28c0a91e 100644 --- a/gdb/doc/ChangeLog +++ b/gdb/doc/ChangeLog @@ -1,3 +1,8 @@ +2013-11-29 Pedro Alves + + * gdb.texinfo (Frames In Python) : + Update comment. + 2013-11-24 Yao Qi * gdb.texinfo (Caching Remote Data): Document new diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index 004c3761bfa..ddb4b38b75c 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -26556,7 +26556,9 @@ function to a string. The value can be one of: No particular reason (older frames should be available). @item gdb.FRAME_UNWIND_NULL_ID -The previous frame's analyzer returns an invalid result. +The previous frame's analyzer returns an invalid result. This is no +longer used by @value{GDBN}, and is kept only for backward +compatibility. @item gdb.FRAME_UNWIND_OUTERMOST This frame is the outermost. diff --git a/gdb/unwind_stop_reasons.def b/gdb/unwind_stop_reasons.def index ca5a74a9f02..2c3d3419d72 100644 --- a/gdb/unwind_stop_reasons.def +++ b/gdb/unwind_stop_reasons.def @@ -31,13 +31,9 @@ or we didn't fail. */ SET (UNWIND_NO_REASON, "no reason") -/* The previous frame's analyzer returns an invalid result - from this_id. - - FIXME drow/2006-08-16: This is how GDB used to indicate end of - stack. We should migrate to a model where frames always have a - valid ID, and this becomes not just an error but an internal - error. But that's a project for another day. */ +/* This is no longer used anywhere, but it's kept because it's exposed + to Python. This is how GDB used to indicate end of stack. We've + now migrated to a model where frames always have a valid ID. */ SET (UNWIND_NULL_ID, "unwinder did not report frame ID") /* This frame is the outermost. */