From eb390f499b820b74e0ab07162b0b8bd12da6f35a Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Wed, 13 Nov 2019 16:33:21 -0700 Subject: [PATCH] Change tui_show_frame_info to return bool This changes tui_show_frame_info to return bool. gdb/ChangeLog 2019-12-20 Tom Tromey * tui/tui-stack.h (tui_show_frame_info): Return bool. * tui/tui-stack.c (tui_show_frame_info): Return bool. * tui/tui-hooks.c (tui_refresh_frame_and_register_information): Update. Change-Id: Id1374f04f919c30a9f50c1beeb70cbc10b9a8f3b --- gdb/ChangeLog | 7 +++++++ gdb/tui/tui-hooks.c | 2 +- gdb/tui/tui-stack.c | 16 +++++++--------- gdb/tui/tui-stack.h | 2 +- 4 files changed, 16 insertions(+), 11 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index c34d1126ec..fa17958118 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2019-12-20 Tom Tromey + + * tui/tui-stack.h (tui_show_frame_info): Return bool. + * tui/tui-stack.c (tui_show_frame_info): Return bool. + * tui/tui-hooks.c (tui_refresh_frame_and_register_information): + Update. + 2019-12-20 Tom Tromey PR tui/18932: diff --git a/gdb/tui/tui-hooks.c b/gdb/tui/tui-hooks.c index deb10b0d37..8576bb8fcc 100644 --- a/gdb/tui/tui-hooks.c +++ b/gdb/tui/tui-hooks.c @@ -133,7 +133,7 @@ tui_refresh_frame_and_register_information () /* Display the frame position (even if there is no symbols or the PC is not known). */ - int frame_info_changed_p = tui_show_frame_info (fi); + bool frame_info_changed_p = tui_show_frame_info (fi); /* Refresh the register window if it's visible. */ if (tui_is_window_visible (DATA_WIN) diff --git a/gdb/tui/tui-stack.c b/gdb/tui/tui-stack.c index d244343859..3f606cbeb4 100644 --- a/gdb/tui/tui-stack.c +++ b/gdb/tui/tui-stack.c @@ -301,10 +301,10 @@ tui_update_locator_fullname (struct symtab *symtab) /* Function to print the frame information for the TUI. The windows are refreshed only if frame information has changed since the last refresh. - Return 1 if frame information has changed (and windows subsequently - refreshed), 0 otherwise. */ + Return true if frame information has changed (and windows + subsequently refreshed), false otherwise. */ -int +bool tui_show_frame_info (struct frame_info *fi) { bool locator_changed_p; @@ -329,15 +329,13 @@ tui_show_frame_info (struct frame_info *fi) not changed. If frame information has not changed, then the windows' contents will not change. So don't bother refreshing the windows. */ if (!locator_changed_p) - return 0; + return false; for (struct tui_source_window_base *win_info : tui_source_windows ()) { win_info->maybe_update (fi, sal); win_info->update_exec_info (); } - - return 1; } else { @@ -346,13 +344,13 @@ tui_show_frame_info (struct frame_info *fi) locator_changed_p = locator->set_locator_info (NULL, sal, ""); if (!locator_changed_p) - return 0; + return false; for (struct tui_source_window_base *win_info : tui_source_windows ()) win_info->erase_source_content (); - - return 1; } + + return true; } void diff --git a/gdb/tui/tui-stack.h b/gdb/tui/tui-stack.h index 3bfe529244..0a928f328b 100644 --- a/gdb/tui/tui-stack.h +++ b/gdb/tui/tui-stack.h @@ -78,6 +78,6 @@ private: extern void tui_update_locator_fullname (struct symtab *symtab); extern void tui_show_locator_content (void); -extern int tui_show_frame_info (struct frame_info *); +extern bool tui_show_frame_info (struct frame_info *); #endif /* TUI_TUI_STACK_H */ -- 2.34.1