Toolchain/binutils-gdb
1
0
Fork 0
forked from Imagelibrary/binutils-gdb
Code Pull Requests Activity

gdb: make frame_unwind_try_unwinder return bool

Browse Source 
Before this commit, the function frame_unwind_try_unwinder would return
an int, where 1 meant the unwinder works, and 0 it doesn't. This is just
a boolean with extra steps, so this commit updates the function to
return bool instead.
This commit is contained in:
Guinevere Larsen
2024-10-14 08:58:29 -03:00
parent 22c6209285
commit a227513b8b
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
gdb/frame-unwind.c
View File

@@ -119,10 +119,10 @@ frame_unwind_append_unwinder (struct gdbarch *gdbarch,
}
/* Call SNIFFER from UNWINDER. If it succeeded set UNWINDER for
THIS_FRAME and return 1. Otherwise the function keeps THIS_FRAME
unchanged and returns 0. */
THIS_FRAME and return true. Otherwise the function keeps THIS_FRAME
unchanged and returns false. */
static int
static bool
frame_unwind_try_unwinder (const frame_info_ptr &this_frame, void **this_cache,
const struct frame_unwind *unwinder)
{
@@ -157,7 +157,7 @@ frame_unwind_try_unwinder (const frame_info_ptr &this_frame, void **this_cache,
thus most unwinders aren't able to determine if they're
the best fit. Keep trying. Fallback prologue unwinders
should always accept the frame. */
return 0;
return false;
}
throw;
}
@@ -165,7 +165,7 @@ frame_unwind_try_unwinder (const frame_info_ptr &this_frame, void **this_cache,
if (res)
{
frame_debug_printf ("yes");
return 1;
return true;
}
else
{
@@ -173,7 +173,7 @@ frame_unwind_try_unwinder (const frame_info_ptr &this_frame, void **this_cache,
/* Don't set *THIS_CACHE to NULL here, because sniffer has to do
so. */
frame_cleanup_after_sniffer (this_frame);
return 0;
return false;
}
gdb_assert_not_reached ("frame_unwind_try_unwinder");
}