public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Use bool for visibility
@ 2019-06-25 13:59 Tom Tromey
0 siblings, 0 replies; only message in thread
From: Tom Tromey @ 2019-06-25 13:59 UTC (permalink / raw)
To: gdb-cvs
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=56122977c04496325ca9d83231e7751ab94d6b14
commit 56122977c04496325ca9d83231e7751ab94d6b14
Author: Tom Tromey <tom@tromey.com>
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 <tom@tromey.com>
* 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) <is_visible>: Now
bool.
(struct tui_win_info, struct tui_source_window_base)
(struct tui_cmd_window) <make_visible>: 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 <tom@tromey.com>
+ * 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) <is_visible>: Now
+ bool.
+ (struct tui_win_info, struct tui_source_window_base)
+ (struct tui_cmd_window) <make_visible>: Change parameter to bool.
+ * tui/tui-data.c (tui_init_generic_part): Update.
+
+2019-06-25 Tom Tromey <tom@tromey.com>
+
* 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);
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2019-06-25 13:59 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-25 13:59 [binutils-gdb] Use bool for visibility Tom Tromey
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).