From patchwork Sun Jun 23 22:42:54 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom Tromey X-Patchwork-Id: 33324 Received: (qmail 59591 invoked by alias); 23 Jun 2019 23:26:12 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 59407 invoked by uid 89); 23 Jun 2019 23:26:10 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-22.2 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=1887, Origin, 369 X-HELO: gateway24.websitewelcome.com Received: from gateway24.websitewelcome.com (HELO gateway24.websitewelcome.com) (192.185.50.45) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sun, 23 Jun 2019 23:26:08 +0000 Received: from cm16.websitewelcome.com (cm16.websitewelcome.com [100.42.49.19]) by gateway24.websitewelcome.com (Postfix) with ESMTP id 2AEF02D5B6 for ; Sun, 23 Jun 2019 18:26:07 -0500 (CDT) Received: from box5379.bluehost.com ([162.241.216.53]) by cmsmtp with SMTP id fBs7hMCV34FKpfBs7hcnTh; Sun, 23 Jun 2019 18:26:07 -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=5r1vyEbNon+EuQ6hWFB2app8DHhkrJuOvmPk11EbszI=; b=BXQawyI1AixSAgX+i2KJsxFSol 57fQ9AjcmC7lBKfGxpeTcTv+ZiVhWhOFT2y6C5KDlQqxt6TT+zGMOxwEC7j317o3hCRLs0/hDq+jT 9kLUcMlnZZqh6I8JsyehpstFs; Received: from 75-166-12-78.hlrn.qwest.net ([75.166.12.78]:54396 helo=bapiya.Home) by box5379.bluehost.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.92) (envelope-from ) id 1hfBD8-000vDQ-EZ; Sun, 23 Jun 2019 17:43:46 -0500 From: Tom Tromey To: gdb-patches@sourceware.org Cc: Tom Tromey Subject: [PATCH 31/66] Use new and delete for tui_gen_win_info Date: Sun, 23 Jun 2019 16:42:54 -0600 Message-Id: <20190623224329.16060-32-tom@tromey.com> In-Reply-To: <20190623224329.16060-1-tom@tromey.com> References: <20190623224329.16060-1-tom@tromey.com> This changes tui_gen_win_info to be allocated with new and destroyed with delete. gdb/ChangeLog 2019-06-23 Tom Tromey * tui/tui-layout.c (init_and_make_win): Use new. * tui/tui-data.h (struct tui_gen_win_info): Add constructor, destructor, initializers. (tui_alloc_generic_win_info): Don't declare. * tui/tui-data.c (_locator): Add argument to constructor. (source_win, disasm_win): New globals. (exec_info): Remove. (tui_source_exec_info_win_ptr, tui_disassem_exec_info_win_ptr): Update. (tui_alloc_generic_win_info): Remove. (init_content_element): Use new. (tui_win_info::tui_win_info): Update. (free_content_elements) : Use delete. --- gdb/ChangeLog | 16 +++++++++++++++ gdb/tui/tui-data.c | 29 ++++++++-------------------- gdb/tui/tui-data.h | 46 +++++++++++++++++++++++++++++++------------- gdb/tui/tui-layout.c | 2 +- 4 files changed, 58 insertions(+), 35 deletions(-) diff --git a/gdb/tui/tui-data.c b/gdb/tui/tui-data.c index af571450316..82af0c9cf77 100644 --- a/gdb/tui/tui-data.c +++ b/gdb/tui/tui-data.c @@ -36,8 +36,9 @@ struct tui_win_info *tui_win_list[MAX_MAJOR_WINDOWS]; ****************************/ static enum tui_layout_type current_layout = UNDEFINED_LAYOUT; static int term_height, term_width; -static struct tui_gen_win_info _locator; -static struct tui_gen_win_info exec_info[2]; +static struct tui_gen_win_info _locator (LOCATOR_WIN); +static struct tui_gen_win_info source_win (EXEC_INFO_WIN); +static struct tui_gen_win_info disasm_win (EXEC_INFO_WIN); static struct std::vector source_windows; static struct tui_win_info *win_with_focus = NULL; static struct tui_layout_def layout_def = { @@ -188,7 +189,7 @@ tui_data_window::clear_detail () struct tui_gen_win_info * tui_source_exec_info_win_ptr (void) { - return &exec_info[0]; + return &source_win; } @@ -196,7 +197,7 @@ tui_source_exec_info_win_ptr (void) struct tui_gen_win_info * tui_disassem_exec_info_win_ptr (void) { - return &exec_info[1]; + return &disasm_win; } @@ -389,17 +390,6 @@ tui_initialize_static_data (void) } -struct tui_gen_win_info * -tui_alloc_generic_win_info (void) -{ - struct tui_gen_win_info *win = XNEW (struct tui_gen_win_info); - - tui_init_generic_part (win); - - return win; -} - - void tui_init_generic_part (struct tui_gen_win_info *win) { @@ -435,9 +425,7 @@ init_content_element (struct tui_win_element *element, element->which_element.source.has_break = FALSE; break; case DATA_WIN: - element->which_element.data_window = XNEW (struct tui_gen_win_info); - tui_init_generic_part (element->which_element.data_window); - element->which_element.data_window->type = DATA_ITEM_WIN; + element->which_element.data_window = new struct tui_gen_win_info (DATA_ITEM_WIN); element->which_element.data_window->content = tui_alloc_content (1, DATA_ITEM_WIN); element->which_element.data_window->content_size = 1; @@ -469,9 +457,8 @@ init_content_element (struct tui_win_element *element, } tui_win_info::tui_win_info (enum tui_win_type type) + : generic (type) { - generic.type = type; - tui_init_generic_part (&generic); } tui_source_window_base::tui_source_window_base (enum tui_win_type type) @@ -711,7 +698,7 @@ free_content_elements (tui_win_content content, xfree (element->which_element.source.line); break; case DATA_WIN: - xfree (element->which_element.data_window); + delete element->which_element.data_window; xfree (element); break; case DATA_ITEM_WIN: diff --git a/gdb/tui/tui-data.h b/gdb/tui/tui-data.h index f915d209782..1fc9e32c5c5 100644 --- a/gdb/tui/tui-data.h +++ b/gdb/tui/tui-data.h @@ -39,18 +39,39 @@ typedef struct tui_win_element **tui_win_content; /* Generic window information. */ struct tui_gen_win_info { - WINDOW *handle; /* Window handle. */ - enum tui_win_type type; /* Type of window. */ - int width; /* Window width. */ - int height; /* Window height. */ - struct tui_point origin; /* Origin of window. */ - tui_win_content content; /* Content of window. */ - int content_size; /* Size of content (# of elements). */ - 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. */ - bool is_visible; /* Whether the window is visible or not. */ - char *title; /* Window title to display. */ + explicit tui_gen_win_info (enum tui_win_type t) + : type (t) + { + } + + ~tui_gen_win_info () + { + } + + /* Window handle. */ + WINDOW *handle = nullptr; + /* Type of window. */ + enum tui_win_type type; + /* Window width. */ + int width = 0; + /* Window height. */ + int height = 0; + /* Origin of window. */ + struct tui_point origin = {0, 0}; + /* Content of window. */ + tui_win_content content = nullptr; + /* Size of content (# of elements). */ + int content_size = 0; + /* Can it be used, or is it already used? */ + int content_in_use = FALSE; + /* Viewport height. */ + int viewport_height = 0; + /* Index of last visible line. */ + int last_visible_line = 0; + /* Whether the window is visible or not. */ + bool is_visible = false; + /* Window title to display. */ + char *title = nullptr; }; /* Constant definitions. */ @@ -465,7 +486,6 @@ extern struct tui_win_info *tui_win_list[MAX_MAJOR_WINDOWS]; /* Data Manipulation Functions. */ extern void tui_initialize_static_data (void); -extern struct tui_gen_win_info *tui_alloc_generic_win_info (void); extern struct tui_win_info *tui_alloc_win_info (enum tui_win_type); extern void tui_init_generic_part (struct tui_gen_win_info *); extern tui_win_content tui_alloc_content (int, enum tui_win_type); diff --git a/gdb/tui/tui-layout.c b/gdb/tui/tui-layout.c index d060c5c7807..556bef833a3 100644 --- a/gdb/tui/tui-layout.c +++ b/gdb/tui/tui-layout.c @@ -822,7 +822,7 @@ init_and_make_win (void *opaque_win_info, if (opaque_win_info == NULL) { if (tui_win_is_auxillary (win_type)) - opaque_win_info = (void *) tui_alloc_generic_win_info (); + opaque_win_info = (void *) new tui_gen_win_info (win_type); else opaque_win_info = (void *) tui_alloc_win_info (win_type); }