From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 111586 invoked by alias); 4 Jul 2019 17:04:07 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 107416 invoked by uid 89); 4 Jul 2019 17:03:41 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-19.3 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,RCVD_IN_DNSWL_NONE,SPF_HELO_PASS autolearn=ham version=3.3.1 spammy= X-HELO: gateway23.websitewelcome.com Received: from gateway23.websitewelcome.com (HELO gateway23.websitewelcome.com) (192.185.49.104) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 04 Jul 2019 17:03:33 +0000 Received: from cm11.websitewelcome.com (cm11.websitewelcome.com [100.42.49.5]) by gateway23.websitewelcome.com (Postfix) with ESMTP id 5263311B3E for ; Thu, 4 Jul 2019 12:03:32 -0500 (CDT) Received: from box5379.bluehost.com ([162.241.216.53]) by cmsmtp with SMTP id j58uhYBeEdnCej58uh6HpS; Thu, 04 Jul 2019 12:03:32 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=tromey.com; s=default; h=References:In-Reply-To:Message-Id:Date:Subject:Cc:To:From: Sender:Reply-To:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=Xq+WLyuPVOThVi7cYFwJPgDdLrLMQmbTm5OWGPVAax0=; b=GWmJdXLj9gXH0XtLYrUYRlKp6i cuekH9gppwN3DdP79/t8ZvYfzeoDgq0xLXWRvDtsk5cDzca0F2kbbHrfFgAx0JBY/Z1lQ12VyRXYp cytuzVPR9IYPlZ4T5JB8vexDr; Received: from 174-29-58-150.hlrn.qwest.net ([174.29.58.150]:34702 helo=bapiya.Home) by box5379.bluehost.com with esmtpa (Exim 4.92) (envelope-from ) id 1hj58u-002sKm-1A; Thu, 04 Jul 2019 12:03:32 -0500 From: Tom Tromey To: gdb-patches@sourceware.org Cc: Tom Tromey Subject: [PATCH 49/61] Remove tui_make_visible and tui_make_invisible Date: Thu, 04 Jul 2019 17:04:00 -0000 Message-Id: <20190704170311.15982-50-tom@tromey.com> In-Reply-To: <20190704170311.15982-1-tom@tromey.com> References: <20190704170311.15982-1-tom@tromey.com> X-SW-Source: 2019-07/txt/msg00156.txt.bz2 tui_make_visible and tui_make_invisible are just wrappers for a method call, so remove them and have the callers simply make the method call themselves. 2019-07-04 Tom Tromey * tui/tui-wingeneral.h (tui_make_visible, tui_make_invisible): Don't declare. * tui/tui-wingeneral.c (tui_make_visible, tui_make_invisible): Remove. * tui/tui-win.c (tui_source_window_base::set_new_height) (tui_source_window_base::set_new_height) (make_invisible_and_set_new_height) (tui_source_window_base::do_make_visible_with_new_height) (tui_source_window_base::do_make_visible_with_new_height): Update. * tui/tui-layout.c (show_source_disasm_command, show_data) (show_source_or_disasm_and_command): Update. * tui/tui-layout.c (show_layout): Update. --- gdb/ChangeLog | 16 ++++++++++++++++ gdb/tui/tui-layout.c | 16 ++++++++-------- gdb/tui/tui-win.c | 10 +++++----- gdb/tui/tui-wingeneral.c | 12 ------------ gdb/tui/tui-wingeneral.h | 2 -- 5 files changed, 29 insertions(+), 27 deletions(-) diff --git a/gdb/tui/tui-layout.c b/gdb/tui/tui-layout.c index 7478c2c966a..4869cd9a11a 100644 --- a/gdb/tui/tui-layout.c +++ b/gdb/tui/tui-layout.c @@ -80,7 +80,7 @@ show_layout (enum tui_layout_type layout) { /* First make the current layout be invisible. */ tui_make_all_invisible (); - tui_make_invisible (tui_locator_win_info_ptr ()); + tui_locator_win_info_ptr ()->make_visible (false); switch (layout) { @@ -514,7 +514,7 @@ show_source_disasm_command (void) tui_term_width (), 0, 0); - tui_make_visible (TUI_SRC_WIN); + TUI_SRC_WIN->make_visible (true); TUI_SRC_WIN->m_has_locator = false; struct tui_locator_window *locator = tui_locator_win_info_ptr (); @@ -527,14 +527,14 @@ show_source_disasm_command (void) tui_term_width (), 0, src_height - 1); - tui_make_visible (TUI_DISASM_WIN); + TUI_DISASM_WIN->make_visible (true); locator->reset (2 /* 1 */ , tui_term_width (), 0, (src_height + asm_height) - 1); TUI_SRC_WIN->m_has_locator = false; TUI_DISASM_WIN->m_has_locator = true; - tui_make_visible (locator); + locator->make_visible (true); tui_show_locator_content (); tui_show_source_content (TUI_DISASM_WIN); @@ -567,7 +567,7 @@ show_data (enum tui_layout_type new_layout) data_height = total_height / 2; src_height = total_height - data_height; tui_make_all_invisible (); - tui_make_invisible (locator); + locator->make_visible (false); if (tui_win_list[DATA_WIN] == nullptr) tui_win_list[DATA_WIN] = new tui_data_window (); tui_win_list[DATA_WIN]->reset (data_height, tui_term_width (), 0, 0); @@ -598,7 +598,7 @@ show_data (enum tui_layout_type new_layout) total_height - 1); base->make_visible (true); base->m_has_locator = true; - tui_make_visible (locator); + locator->make_visible (true); tui_show_locator_content (); tui_add_to_source_windows (base); tui_set_current_layout_to (new_layout); @@ -662,11 +662,11 @@ show_source_or_disasm_and_command (enum tui_layout_type layout_type) tui_term_width (), 0, 0); - tui_make_visible (win_info); + win_info->make_visible (true); win_info->m_has_locator = true; - tui_make_visible (locator); + locator->make_visible (true); tui_show_locator_content (); tui_show_source_content (win_info); diff --git a/gdb/tui/tui-win.c b/gdb/tui/tui-win.c index 9d9d9a5f577..c92a706efc6 100644 --- a/gdb/tui/tui-win.c +++ b/gdb/tui/tui-win.c @@ -1226,7 +1226,7 @@ tui_adjust_win_heights (struct tui_win_info *primary_win_info, void tui_source_window_base::set_new_height (int height) { - tui_make_invisible (execution_info); + execution_info->make_visible (false); execution_info->height = height; execution_info->origin.y = origin.y; if (height > 1) @@ -1238,7 +1238,7 @@ tui_source_window_base::set_new_height (int height) if (has_locator ()) { tui_locator_window *gen_win_info = tui_locator_win_info_ptr (); - tui_make_invisible (gen_win_info); + gen_win_info->make_visible (false); gen_win_info->origin.y = origin.y + height; } } @@ -1263,7 +1263,7 @@ static void make_invisible_and_set_new_height (struct tui_win_info *win_info, int height) { - tui_make_invisible (win_info); + win_info->make_visible (false); win_info->height = height; if (height > 1) win_info->viewport_height = height - 1; @@ -1292,7 +1292,7 @@ tui_win_info::make_visible_with_new_height () void tui_source_window_base::do_make_visible_with_new_height () { - tui_make_visible (execution_info); + execution_info->make_visible (true); if (!content.empty ()) { struct tui_line_or_address line_or_addr; @@ -1326,7 +1326,7 @@ tui_source_window_base::do_make_visible_with_new_height () } if (has_locator ()) { - tui_make_visible (tui_locator_win_info_ptr ()); + tui_locator_win_info_ptr ()->make_visible (true); tui_show_locator_content (); } } diff --git a/gdb/tui/tui-wingeneral.c b/gdb/tui/tui-wingeneral.c index 4e6637e8687..2fb8d6c684f 100644 --- a/gdb/tui/tui-wingeneral.c +++ b/gdb/tui/tui-wingeneral.c @@ -179,18 +179,6 @@ tui_gen_win_info::make_visible (bool visible) } } -void -tui_make_visible (struct tui_gen_win_info *win_info) -{ - win_info->make_visible (true); -} - -void -tui_make_invisible (struct tui_gen_win_info *win_info) -{ - win_info->make_visible (false); -} - /* See tui-data.h. */ void diff --git a/gdb/tui/tui-wingeneral.h b/gdb/tui/tui-wingeneral.h index 20b7f21c7c0..5e6198d4366 100644 --- a/gdb/tui/tui-wingeneral.h +++ b/gdb/tui/tui-wingeneral.h @@ -28,8 +28,6 @@ struct tui_win_info; struct tui_gen_win_info; extern void tui_unhighlight_win (struct tui_win_info *); -extern void tui_make_visible (struct tui_gen_win_info *); -extern void tui_make_invisible (struct tui_gen_win_info *); extern void tui_make_all_visible (void); extern void tui_make_all_invisible (void); extern void tui_make_window (struct tui_gen_win_info *, enum tui_box); -- 2.17.2