public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Introduce set_highlight method
@ 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=214a5cbea656f3ffc1e7e525627c56849347c709
commit 214a5cbea656f3ffc1e7e525627c56849347c709
Author: Tom Tromey <tom@tromey.com>
Date: Sun Jun 16 15:28:03 2019 -0600
Introduce set_highlight method
This introduces the tui_win_info::set_highlight method, and changes
the highlighting-related code to use bool rather than int.
gdb/ChangeLog
2019-06-25 Tom Tromey <tom@tromey.com>
* tui/tui-wingeneral.c (tui_unhighlight_win, tui_highlight_win):
Update.
* tui/tui-layout.c (make_command_window)
(show_source_disasm_command, show_data, init_and_make_win)
(show_source_or_disasm_and_command): Update.
* tui/tui-data.h (struct tui_win_info) <set_highlight>: New
method.
<can_highight, is_highlighted>: Now bool.
(tui_set_win_highlight): Don't declare.
* tui/tui-data.c (tui_set_win_highlight): Remove.
Diff:
---
gdb/ChangeLog | 13 +++++++++++++
gdb/tui/tui-data.c | 8 --------
gdb/tui/tui-data.h | 12 ++++++++----
gdb/tui/tui-layout.c | 18 +++++++++---------
gdb/tui/tui-wingeneral.c | 4 ++--
5 files changed, 32 insertions(+), 23 deletions(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 0d66374..71fe526 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,18 @@
2019-06-25 Tom Tromey <tom@tromey.com>
+ * tui/tui-wingeneral.c (tui_unhighlight_win, tui_highlight_win):
+ Update.
+ * tui/tui-layout.c (make_command_window)
+ (show_source_disasm_command, show_data, init_and_make_win)
+ (show_source_or_disasm_and_command): Update.
+ * tui/tui-data.h (struct tui_win_info) <set_highlight>: New
+ method.
+ <can_highight, is_highlighted>: Now bool.
+ (tui_set_win_highlight): Don't declare.
+ * tui/tui-data.c (tui_set_win_highlight): Remove.
+
+2019-06-25 Tom Tromey <tom@tromey.com>
+
* tui/tui-wingeneral.c (make_visible): Remove check of window
type.
diff --git a/gdb/tui/tui-data.c b/gdb/tui/tui-data.c
index 3298eb8..a351113 100644
--- a/gdb/tui/tui-data.c
+++ b/gdb/tui/tui-data.c
@@ -69,14 +69,6 @@ tui_win_is_auxillary (enum tui_win_type win_type)
return (win_type > MAX_MAJOR_WINDOWS);
}
-void
-tui_set_win_highlight (struct tui_win_info *win_info,
- int highlight)
-{
- if (win_info != NULL)
- win_info->is_highlighted = highlight;
-}
-
/******************************************
** ACCESSORS & MUTATORS FOR PRIVATE DATA
******************************************/
diff --git a/gdb/tui/tui-data.h b/gdb/tui/tui-data.h
index 08b0dad..572fea5 100644
--- a/gdb/tui/tui-data.h
+++ b/gdb/tui/tui-data.h
@@ -276,6 +276,12 @@ public:
/* Compute the maximum height of this window. */
virtual int max_height () const;
+ /* Set whether this window is highglighted. */
+ void set_highlight (bool highlight)
+ {
+ is_highlighted = highlight;
+ }
+
/* Methods to scroll the contents of this window. Note that they
are named with "_scroll" coming at the end because the more
obvious "scroll_forward" is defined as a macro in term.h. */
@@ -287,10 +293,10 @@ public:
struct tui_gen_win_info generic; /* General window information. */
/* Can this window ever be highlighted? */
- int can_highlight = 0;
+ bool can_highlight = false;
/* Is this window highlighted? */
- int is_highlighted = 0;
+ bool is_highlighted = false;
};
/* The base class for all source-like windows, namely the source and
@@ -447,8 +453,6 @@ protected:
};
extern int tui_win_is_auxillary (enum tui_win_type win_type);
-extern void tui_set_win_highlight (struct tui_win_info *win_info,
- int highlight);
/* Global Data. */
diff --git a/gdb/tui/tui-layout.c b/gdb/tui/tui-layout.c
index 9b2bd0b..d060c5c 100644
--- a/gdb/tui/tui-layout.c
+++ b/gdb/tui/tui-layout.c
@@ -546,7 +546,7 @@ make_command_window (int height, int origin_y)
0,
origin_y,
DONT_BOX_WINDOW);
- result->can_highlight = FALSE;
+ result->can_highlight = false;
return result;
}
@@ -627,7 +627,7 @@ show_source_disasm_command (void)
TUI_SRC_WIN->generic.width,
TUI_SRC_WIN->execution_info->width,
0);
- TUI_SRC_WIN->can_highlight = TRUE;
+ TUI_SRC_WIN->can_highlight = true;
init_gen_win_info (TUI_SRC_WIN->execution_info,
EXEC_INFO_WIN,
src_height,
@@ -677,7 +677,7 @@ show_source_disasm_command (void)
3,
0,
src_height - 1);
- TUI_DISASM_WIN->can_highlight = TRUE;
+ TUI_DISASM_WIN->can_highlight = true;
tui_make_visible (&TUI_DISASM_WIN->generic);
tui_make_visible (TUI_DISASM_WIN->execution_info);
}
@@ -698,7 +698,7 @@ show_source_disasm_command (void)
TUI_CMD_WIN->generic.width,
0,
TUI_CMD_WIN->generic.origin.y);
- TUI_CMD_WIN->can_highlight = FALSE;
+ TUI_CMD_WIN->can_highlight = false;
tui_make_visible (&TUI_CMD_WIN->generic);
}
tui_refresh_win (&TUI_CMD_WIN->generic);
@@ -723,7 +723,7 @@ show_data (enum tui_layout_type new_layout)
tui_make_all_invisible ();
tui_make_invisible (locator);
make_data_window (&tui_win_list[DATA_WIN], data_height, 0);
- TUI_DATA_WIN->can_highlight = TRUE;
+ TUI_DATA_WIN->can_highlight = true;
if (new_layout == SRC_DATA_COMMAND)
win_type = SRC_WIN;
else
@@ -835,9 +835,9 @@ init_and_make_win (void *opaque_win_info,
if (!tui_win_is_auxillary (win_type))
{
if (generic->type == CMD_WIN)
- ((struct tui_win_info *) opaque_win_info)->can_highlight = FALSE;
+ ((struct tui_win_info *) opaque_win_info)->can_highlight = false;
else
- ((struct tui_win_info *) opaque_win_info)->can_highlight = TRUE;
+ ((struct tui_win_info *) opaque_win_info)->can_highlight = true;
}
tui_make_window (generic, box_it);
@@ -942,7 +942,7 @@ show_source_or_disasm_and_command (enum tui_layout_type layout_type)
3,
0,
0);
- base->can_highlight = TRUE;
+ base->can_highlight = true;
tui_make_visible (&(*win_info_ptr)->generic);
tui_make_visible (base->execution_info);
}
@@ -966,7 +966,7 @@ show_source_or_disasm_and_command (enum tui_layout_type layout_type)
TUI_CMD_WIN->generic.width,
TUI_CMD_WIN->generic.origin.x,
TUI_CMD_WIN->generic.origin.y);
- TUI_CMD_WIN->can_highlight = FALSE;
+ TUI_CMD_WIN->can_highlight = false;
tui_make_visible (&TUI_CMD_WIN->generic);
}
tui_set_current_layout_to (layout_type);
diff --git a/gdb/tui/tui-wingeneral.c b/gdb/tui/tui-wingeneral.c
index 73d77ce..c177828 100644
--- a/gdb/tui/tui-wingeneral.c
+++ b/gdb/tui/tui-wingeneral.c
@@ -114,7 +114,7 @@ tui_unhighlight_win (struct tui_win_info *win_info)
{
box_win (&win_info->generic, NO_HILITE);
wrefresh (win_info->generic.handle);
- tui_set_win_highlight (win_info, 0);
+ win_info->set_highlight (false);
}
}
@@ -128,7 +128,7 @@ tui_highlight_win (struct tui_win_info *win_info)
{
box_win (&win_info->generic, HILITE);
wrefresh (win_info->generic.handle);
- tui_set_win_highlight (win_info, 1);
+ win_info->set_highlight (true);
}
}
^ 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] Introduce set_highlight method 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).