aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Tromey <tom@tromey.com>2020-02-22 11:48:26 -0700
committerTom Tromey <tom@tromey.com>2020-02-22 11:48:38 -0700
commit935c78c0468215e2f034f39b8285fa8bb17729b8 (patch)
treed40264042515b73c2bb8ebc02f98b937c00c23d0
parent0240c8f11b06a33a043bacbe2b9e723ae6ad573c (diff)
downloadbinutils-935c78c0468215e2f034f39b8285fa8bb17729b8.zip
binutils-935c78c0468215e2f034f39b8285fa8bb17729b8.tar.gz
binutils-935c78c0468215e2f034f39b8285fa8bb17729b8.tar.bz2
Remove tui_set_win_with_focus
I noticed that the TUI had two functions with similar names: tui_set_win_focus_to and tui_set_win_with_focus. However, one was just an implementation detail of the latter. So, this patch removes tui_set_win_with_focus entirely, to avoid any temptation to call it. gdb/ChangeLog 2020-02-22 Tom Tromey <tom@tromey.com> * tui/tui-win.c (tui_set_win_focus_to): Move to tui-data.c. * tui/tui-data.h (tui_set_win_with_focus): Don't declare. * tui/tui-data.c (tui_set_win_with_focus): Remove. (tui_set_win_focus_to): Move from tui-win.c. Change-Id: Idffddab773436bdf80d55480906d76b292981ef2
-rw-r--r--gdb/ChangeLog7
-rw-r--r--gdb/tui/tui-data.c12
-rw-r--r--gdb/tui/tui-data.h1
-rw-r--r--gdb/tui/tui-win.c15
4 files changed, 16 insertions, 19 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index fe86927..474825b 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,12 @@
2020-02-22 Tom Tromey <tom@tromey.com>
+ * tui/tui-win.c (tui_set_win_focus_to): Move to tui-data.c.
+ * tui/tui-data.h (tui_set_win_with_focus): Don't declare.
+ * tui/tui-data.c (tui_set_win_with_focus): Remove.
+ (tui_set_win_focus_to): Move from tui-win.c.
+
+2020-02-22 Tom Tromey <tom@tromey.com>
+
* tui/tui-layout.c (make_standard_window, get_locator_window): New
functions.
(known_window_types): New global.
diff --git a/gdb/tui/tui-data.c b/gdb/tui/tui-data.c
index db637d8..8f7d257 100644
--- a/gdb/tui/tui-data.c
+++ b/gdb/tui/tui-data.c
@@ -23,6 +23,7 @@
#include "symtab.h"
#include "tui/tui.h"
#include "tui/tui-data.h"
+#include "tui/tui-win.h"
#include "tui/tui-wingeneral.h"
#include "tui/tui-winsource.h"
#include "gdb_curses.h"
@@ -59,11 +60,16 @@ tui_win_with_focus (void)
}
-/* Set the window that has the logical focus. */
+/* Set the logical focus to win_info. */
void
-tui_set_win_with_focus (struct tui_win_info *win_info)
+tui_set_win_focus_to (struct tui_win_info *win_info)
{
- win_with_focus = win_info;
+ if (win_info != NULL)
+ {
+ tui_unhighlight_win (win_with_focus);
+ win_with_focus = win_info;
+ tui_highlight_win (win_info);
+ }
}
diff --git a/gdb/tui/tui-data.h b/gdb/tui/tui-data.h
index 51208b1..60c588e 100644
--- a/gdb/tui/tui-data.h
+++ b/gdb/tui/tui-data.h
@@ -251,7 +251,6 @@ extern int tui_term_width (void);
extern void tui_set_term_width_to (int);
extern struct tui_locator_window *tui_locator_win_info_ptr (void);
extern struct tui_win_info *tui_win_with_focus (void);
-extern void tui_set_win_with_focus (struct tui_win_info *);
extern bool tui_win_resized ();
extern void tui_set_win_resized_to (bool);
diff --git a/gdb/tui/tui-win.c b/gdb/tui/tui-win.c
index ac314d7..51d0365 100644
--- a/gdb/tui/tui-win.c
+++ b/gdb/tui/tui-win.c
@@ -452,21 +452,6 @@ tui_update_gdb_sizes (void)
}
-/* Set the logical focus to win_info. */
-void
-tui_set_win_focus_to (struct tui_win_info *win_info)
-{
- if (win_info != NULL)
- {
- struct tui_win_info *win_with_focus = tui_win_with_focus ();
-
- tui_unhighlight_win (win_with_focus);
- tui_set_win_with_focus (win_info);
- tui_highlight_win (win_info);
- }
-}
-
-
void
tui_win_info::forward_scroll (int num_to_scroll)
{