From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 119466 invoked by alias); 25 Jun 2019 13:59:08 -0000 Mailing-List: contact gdb-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: , Sender: gdb-cvs-owner@sourceware.org List-Subscribe: Sender: gdb-cvs-owner@sourceware.org Received: (qmail 119442 invoked by uid 306); 25 Jun 2019 13:59:08 -0000 Date: Tue, 25 Jun 2019 13:59:00 -0000 Message-ID: <20190625135908.119441.qmail@sourceware.org> Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Tom Tromey To: gdb-cvs@sourceware.org Subject: [binutils-gdb] Use bool for visibility X-Act-Checkin: binutils-gdb X-Git-Author: Tom Tromey X-Git-Refname: refs/heads/master X-Git-Oldrev: cda37efbffb63e2634ea0600fdad2de6a8f9f3ad X-Git-Newrev: 56122977c04496325ca9d83231e7751ab94d6b14 X-SW-Source: 2019-06/txt/msg00158.txt.bz2 https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=56122977c04496325ca9d83231e7751ab94d6b14 commit 56122977c04496325ca9d83231e7751ab94d6b14 Author: Tom Tromey Date: Sun Jun 16 14:24:42 2019 -0600 Use bool for visibility This changes the visibility-related functions and data members in the TUI to use bool rather than int. gdb/ChangeLog 2019-06-25 Tom Tromey * tui/tui.h (tui_is_window_visible): Return bool. * tui/tui.c (tui_is_window_visible): Return bool. * tui/tui-wingeneral.c (tui_make_window, make_visible) (tui_make_visible, tui_make_invisible) (tui_win_info::make_visible) (tui_source_window_base::make_visible, make_all_visible) (tui_make_all_visible, tui_make_all_invisible): Update. * tui/tui-windata.c (tui_delete_data_content_windows): Update. * tui/tui-data.h (struct tui_gen_win_info) : Now bool. (struct tui_win_info, struct tui_source_window_base) (struct tui_cmd_window) : Change parameter to bool. * tui/tui-data.c (tui_init_generic_part): Update. Diff: --- gdb/ChangeLog | 16 ++++++++++++++++ gdb/tui/tui-data.c | 4 ++-- gdb/tui/tui-data.h | 8 ++++---- gdb/tui/tui-windata.c | 2 +- gdb/tui/tui-wingeneral.c | 22 +++++++++++----------- gdb/tui/tui.c | 6 +++--- gdb/tui/tui.h | 2 +- 7 files changed, 38 insertions(+), 22 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index c4e464b..b47b6ce 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,21 @@ 2019-06-25 Tom Tromey + * tui/tui.h (tui_is_window_visible): Return bool. + * tui/tui.c (tui_is_window_visible): Return bool. + * tui/tui-wingeneral.c (tui_make_window, make_visible) + (tui_make_visible, tui_make_invisible) + (tui_win_info::make_visible) + (tui_source_window_base::make_visible, make_all_visible) + (tui_make_all_visible, tui_make_all_invisible): Update. + * tui/tui-windata.c (tui_delete_data_content_windows): Update. + * tui/tui-data.h (struct tui_gen_win_info) : Now + bool. + (struct tui_win_info, struct tui_source_window_base) + (struct tui_cmd_window) : Change parameter to bool. + * tui/tui-data.c (tui_init_generic_part): Update. + +2019-06-25 Tom Tromey + * tui/tui-wingeneral.c (tui_win_info::make_visible) (tui_source_window_base::make_visible): New methods. (make_all_visible): Make method call. diff --git a/gdb/tui/tui-data.c b/gdb/tui/tui-data.c index 6e1df01..6df7ce2 100644 --- a/gdb/tui/tui-data.c +++ b/gdb/tui/tui-data.c @@ -420,8 +420,8 @@ tui_init_generic_part (struct tui_gen_win_info *win) win->last_visible_line = 0; win->handle = NULL; win->content = NULL; - win->content_in_use = - win->is_visible = FALSE; + win->content_in_use = FALSE; + win->is_visible = false; win->title = 0; } diff --git a/gdb/tui/tui-data.h b/gdb/tui/tui-data.h index 4362af0..3799ab4 100644 --- a/gdb/tui/tui-data.h +++ b/gdb/tui/tui-data.h @@ -49,7 +49,7 @@ struct tui_gen_win_info int content_in_use; /* Can it be used, or is it already used? */ int viewport_height; /* Viewport height. */ int last_visible_line; /* Index of last visible line. */ - int is_visible; /* Whether the window is visible or not. */ + bool is_visible; /* Whether the window is visible or not. */ char *title; /* Window title to display. */ }; @@ -256,7 +256,7 @@ public: } /* Make this window visible or invisible. */ - virtual void make_visible (int visible); + virtual void make_visible (bool visible); /* Methods to scroll the contents of this window. Note that they are named with "_scroll" coming at the end because the more @@ -298,7 +298,7 @@ public: return m_has_locator; } - void make_visible (int visible) override; + void make_visible (bool visible) override; /* Does the locator belong to this window? */ bool m_has_locator = false; @@ -393,7 +393,7 @@ struct tui_cmd_window : public tui_win_info void clear_detail () override; - void make_visible (int visible) override + void make_visible (bool visible) override { } diff --git a/gdb/tui/tui-windata.c b/gdb/tui/tui-windata.c index a0f20c3..67228a6 100644 --- a/gdb/tui/tui-windata.c +++ b/gdb/tui/tui-windata.c @@ -78,7 +78,7 @@ tui_delete_data_content_windows (void) = &TUI_DATA_WIN->generic.content[i]->which_element.data_window; tui_delete_win (data_item_win_ptr->handle); data_item_win_ptr->handle = NULL; - data_item_win_ptr->is_visible = FALSE; + data_item_win_ptr->is_visible = false; } } diff --git a/gdb/tui/tui-wingeneral.c b/gdb/tui/tui-wingeneral.c index 1308437..e802e52 100644 --- a/gdb/tui/tui-wingeneral.c +++ b/gdb/tui/tui-wingeneral.c @@ -161,7 +161,7 @@ tui_make_window (struct tui_gen_win_info *win_info, int box_it) { if (box_it == BOX_WINDOW) box_win (win_info, NO_HILITE); - win_info->is_visible = TRUE; + win_info->is_visible = true; scrollok (handle, TRUE); } } @@ -171,7 +171,7 @@ tui_make_window (struct tui_gen_win_info *win_info, int box_it) delete the entire window when making it visible, and create it again when making it visible. */ static void -make_visible (struct tui_gen_win_info *win_info, int visible) +make_visible (struct tui_gen_win_info *win_info, bool visible) { /* Don't tear down/recreate command window. */ if (win_info->type == CMD_WIN) @@ -184,14 +184,14 @@ make_visible (struct tui_gen_win_info *win_info, int visible) tui_make_window (win_info, (win_info->type != CMD_WIN && !tui_win_is_auxillary (win_info->type))); - win_info->is_visible = TRUE; + win_info->is_visible = true; } } else if (!visible && win_info->is_visible && win_info->handle != NULL) { - win_info->is_visible = FALSE; + win_info->is_visible = false; tui_delete_win (win_info->handle); win_info->handle = NULL; } @@ -202,19 +202,19 @@ make_visible (struct tui_gen_win_info *win_info, int visible) void tui_make_visible (struct tui_gen_win_info *win_info) { - make_visible (win_info, 1); + make_visible (win_info, true); } void tui_make_invisible (struct tui_gen_win_info *win_info) { - make_visible (win_info, 0); + make_visible (win_info, false); } /* See tui-data.h. */ void -tui_win_info::make_visible (int visible) +tui_win_info::make_visible (bool visible) { ::make_visible (&generic, visible); } @@ -222,7 +222,7 @@ tui_win_info::make_visible (int visible) /* See tui-data.h. */ void -tui_source_window_base::make_visible (int visible) +tui_source_window_base::make_visible (bool visible) { ::make_visible (execution_info, visible); tui_win_info::make_visible (visible); @@ -231,7 +231,7 @@ tui_source_window_base::make_visible (int visible) /* Makes all windows invisible (except the command and locator windows). */ static void -make_all_visible (int visible) +make_all_visible (bool visible) { int i; @@ -247,13 +247,13 @@ make_all_visible (int visible) void tui_make_all_visible (void) { - make_all_visible (1); + make_all_visible (true); } void tui_make_all_invisible (void) { - make_all_visible (0); + make_all_visible (false); } /* Function to refresh all the windows currently displayed. */ diff --git a/gdb/tui/tui.c b/gdb/tui/tui.c index ce8de72..d720199 100644 --- a/gdb/tui/tui.c +++ b/gdb/tui/tui.c @@ -655,14 +655,14 @@ tui_show_assembly (struct gdbarch *gdbarch, CORE_ADDR addr) tui_update_source_windows_with_addr (gdbarch, addr); } -int +bool tui_is_window_visible (enum tui_win_type type) { if (tui_active == 0) - return 0; + return false; if (tui_win_list[type] == 0) - return 0; + return false; return tui_win_list[type]->generic.is_visible; } diff --git a/gdb/tui/tui.h b/gdb/tui/tui.h index 5b8be0c..8f6ef64 100644 --- a/gdb/tui/tui.h +++ b/gdb/tui/tui.h @@ -56,7 +56,7 @@ enum tui_win_type extern CORE_ADDR tui_get_low_disassembly_address (struct gdbarch *, CORE_ADDR, CORE_ADDR); extern void tui_show_assembly (struct gdbarch *gdbarch, CORE_ADDR addr); -extern int tui_is_window_visible (enum tui_win_type type); +extern bool tui_is_window_visible (enum tui_win_type type); extern int tui_get_command_dimension (unsigned int *width, unsigned int *height);