From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 119700 invoked by alias); 25 Jun 2019 13:59:13 -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 119681 invoked by uid 306); 25 Jun 2019 13:59:13 -0000 Date: Tue, 25 Jun 2019 13:59:00 -0000 Message-ID: <20190625135913.119680.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] Introduce the refresh method X-Act-Checkin: binutils-gdb X-Git-Author: Tom Tromey X-Git-Refname: refs/heads/master X-Git-Oldrev: 56122977c04496325ca9d83231e7751ab94d6b14 X-Git-Newrev: 2042b506c85274b4c652fbc9291be65b2550104d X-SW-Source: 2019-06/txt/msg00159.txt.bz2 https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=2042b506c85274b4c652fbc9291be65b2550104d commit 2042b506c85274b4c652fbc9291be65b2550104d Author: Tom Tromey Date: Sun Jun 16 14:27:28 2019 -0600 Introduce the refresh method This adds tui_win_info::refresh and updates tui_source_window_base to implement it as well. This lets us simplify tui_refresh_all, removing a check of the window type. gdb/ChangeLog 2019-06-25 Tom Tromey * tui/tui-wingeneral.c (tui_win_info::refresh) (tui_source_window_base::refresh): New methods. (tui_refresh_all): Call the refresh method. * tui/tui-data.h (struct tui_win_info) (struct tui_source_window_base) : New method. Diff: --- gdb/ChangeLog | 8 ++++++++ gdb/tui/tui-data.h | 4 ++++ gdb/tui/tui-wingeneral.c | 31 ++++++++++++++++++++----------- 3 files changed, 32 insertions(+), 11 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index b47b6ce..1fea395 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,13 @@ 2019-06-25 Tom Tromey + * tui/tui-wingeneral.c (tui_win_info::refresh) + (tui_source_window_base::refresh): New methods. + (tui_refresh_all): Call the refresh method. + * tui/tui-data.h (struct tui_win_info) + (struct tui_source_window_base) : New method. + +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) diff --git a/gdb/tui/tui-data.h b/gdb/tui/tui-data.h index 3799ab4..e4a9b04 100644 --- a/gdb/tui/tui-data.h +++ b/gdb/tui/tui-data.h @@ -258,6 +258,9 @@ public: /* Make this window visible or invisible. */ virtual void make_visible (bool visible); + /* Refresh this window and any associated windows. */ + virtual void refresh (); + /* 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. */ @@ -299,6 +302,7 @@ public: } void make_visible (bool visible) override; + void refresh () override; /* Does the locator belong to this window? */ bool m_has_locator = false; diff --git a/gdb/tui/tui-wingeneral.c b/gdb/tui/tui-wingeneral.c index e802e52..4d168af 100644 --- a/gdb/tui/tui-wingeneral.c +++ b/gdb/tui/tui-wingeneral.c @@ -256,6 +256,25 @@ tui_make_all_invisible (void) make_all_visible (false); } +/* See tui-data.h. */ + +void +tui_win_info::refresh () +{ + touchwin (generic.handle); + tui_refresh_win (&generic); +} + +/* See tui-data.h. */ + +void +tui_source_window_base::refresh () +{ + touchwin (execution_info->handle); + tui_refresh_win (execution_info); + tui_win_info::refresh (); +} + /* Function to refresh all the windows currently displayed. */ void @@ -267,17 +286,7 @@ tui_refresh_all (struct tui_win_info **list) for (type = SRC_WIN; (type < MAX_MAJOR_WINDOWS); type++) { if (list[type] && list[type]->generic.is_visible) - { - if (type == SRC_WIN || type == DISASSEM_WIN) - { - tui_source_window_base *base - = (tui_source_window_base *) list[type]; - touchwin (base->execution_info->handle); - tui_refresh_win (base->execution_info); - } - touchwin (list[type]->generic.handle); - tui_refresh_win (&list[type]->generic); - } + list[type]->refresh (); } if (locator->is_visible) {