From patchwork Thu Nov 2 22:36:04 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom Tromey X-Patchwork-Id: 24051 Received: (qmail 7595 invoked by alias); 2 Nov 2017 22:36:23 -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 7313 invoked by uid 89); 2 Nov 2017 22:36:20 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-25.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.2 spammy=3888, belonging, rfa X-HELO: gateway33.websitewelcome.com Received: from gateway33.websitewelcome.com (HELO gateway33.websitewelcome.com) (192.185.145.4) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 02 Nov 2017 22:36:17 +0000 Received: from cm17.websitewelcome.com (cm17.websitewelcome.com [100.42.49.20]) by gateway33.websitewelcome.com (Postfix) with ESMTP id 6B2142E44EC for ; Thu, 2 Nov 2017 17:36:16 -0500 (CDT) Received: from box5379.bluehost.com ([162.241.216.53]) by cmsmtp with SMTP id AO5wezYwgc72gAO5wexjF8; Thu, 02 Nov 2017 17:36:16 -0500 Received: from 71-218-90-63.hlrn.qwest.net ([71.218.90.63]:58978 helo=bapiya.Home) by box5379.bluehost.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.89) (envelope-from ) id 1eAO5w-003CfJ-7X; Thu, 02 Nov 2017 17:36:16 -0500 From: Tom Tromey To: gdb-patches@sourceware.org Cc: Tom Tromey Subject: [RFA 05/13] Remove directive-searched cleanups Date: Thu, 2 Nov 2017 16:36:04 -0600 Message-Id: <20171102223612.3642-6-tom@tromey.com> In-Reply-To: <20171102223612.3642-1-tom@tromey.com> References: <20171102223612.3642-1-tom@tromey.com> X-BWhitelist: no X-Source-L: No X-Exim-ID: 1eAO5w-003CfJ-7X X-Source-Sender: 71-218-90-63.hlrn.qwest.net (bapiya.Home) [71.218.90.63]:58978 X-Source-Auth: tom+tromey.com X-Email-Count: 6 X-Source-Cap: ZWx5bnJvYmk7ZWx5bnJvYmk7Ym94NTM3OS5ibHVlaG9zdC5jb20= X-Local-Domain: yes This removes a few cleanups related to the "searched" field in struct using_direct, replacing these with scoped_restore. gdb/ChangeLog 2017-11-02 Tom Tromey * cp-namespace.c (reset_directive_searched): Remove. (cp_lookup_symbol_via_imports): Use scoped_restore. * cp-support.c (reset_directive_searched): Remove. (make_symbol_overload_list_using): Use scoped_restore. * d-namespace.c (d_lookup_symbol_imports): Use scoped_restore. (reset_directive_searched): Remove. --- gdb/ChangeLog | 9 +++++++++ gdb/cp-namespace.c | 25 +++---------------------- gdb/cp-support.c | 19 ++----------------- gdb/d-namespace.c | 26 +++----------------------- 4 files changed, 17 insertions(+), 62 deletions(-) diff --git a/gdb/cp-namespace.c b/gdb/cp-namespace.c index 214b7e1cf4..d8817c0372 100644 --- a/gdb/cp-namespace.c +++ b/gdb/cp-namespace.c @@ -338,15 +338,6 @@ cp_lookup_symbol_in_namespace (const char *the_namespace, const char *name, return sym; } -/* Used for cleanups to reset the "searched" flag in case of an error. */ - -static void -reset_directive_searched (void *data) -{ - struct using_direct *direct = (struct using_direct *) data; - direct->searched = 0; -} - /* Search for NAME by applying all import statements belonging to BLOCK which are applicable in SCOPE. If DECLARATION_ONLY the search is restricted to using declarations. @@ -388,7 +379,6 @@ cp_lookup_symbol_via_imports (const char *scope, struct block_symbol sym; int len; int directive_match; - struct cleanup *searched_cleanup; sym.symbol = NULL; sym.block = NULL; @@ -425,9 +415,8 @@ cp_lookup_symbol_via_imports (const char *scope, { /* Mark this import as searched so that the recursive call does not search it again. */ - current->searched = 1; - searched_cleanup = make_cleanup (reset_directive_searched, - current); + scoped_restore reset_directive_searched + = make_scoped_restore (¤t->searched, 1); /* If there is an import of a single declaration, compare the imported declaration (after optional renaming by its alias) @@ -446,9 +435,6 @@ cp_lookup_symbol_via_imports (const char *scope, search of this import is complete. */ if (declaration_only || sym.symbol != NULL || current->declaration) { - current->searched = 0; - discard_cleanups (searched_cleanup); - if (sym.symbol != NULL) return sym; @@ -460,10 +446,7 @@ cp_lookup_symbol_via_imports (const char *scope, if (strcmp (name, *excludep) == 0) break; if (*excludep) - { - discard_cleanups (searched_cleanup); - continue; - } + continue; if (current->alias != NULL && strcmp (name, current->alias) == 0) @@ -484,8 +467,6 @@ cp_lookup_symbol_via_imports (const char *scope, name, block, domain, 1, 0, 0); } - current->searched = 0; - discard_cleanups (searched_cleanup); if (sym.symbol != NULL) return sym; diff --git a/gdb/cp-support.c b/gdb/cp-support.c index 37b2b4af97..817de59f12 100644 --- a/gdb/cp-support.c +++ b/gdb/cp-support.c @@ -1286,16 +1286,6 @@ make_symbol_overload_list_adl (struct type **arg_types, int nargs, return sym_return_val; } -/* Used for cleanups to reset the "searched" flag in case of an - error. */ - -static void -reset_directive_searched (void *data) -{ - struct using_direct *direct = (struct using_direct *) data; - direct->searched = 0; -} - /* This applies the using directives to add namespaces to search in, and then searches for overloads in all of those namespaces. It adds the symbols found to sym_return_val. Arguments are as in @@ -1332,16 +1322,11 @@ make_symbol_overload_list_using (const char *func_name, { /* Mark this import as searched so that the recursive call does not search it again. */ - struct cleanup *old_chain; - current->searched = 1; - old_chain = make_cleanup (reset_directive_searched, - current); + scoped_restore reset_directive_searched + = make_scoped_restore (¤t->searched, 1); make_symbol_overload_list_using (func_name, current->import_src); - - current->searched = 0; - discard_cleanups (old_chain); } } diff --git a/gdb/d-namespace.c b/gdb/d-namespace.c index bc791f70dc..6842a29fac 100644 --- a/gdb/d-namespace.c +++ b/gdb/d-namespace.c @@ -356,16 +356,6 @@ d_lookup_nested_symbol (struct type *parent_type, } } -/* Used for cleanups to reset the "searched" flag incase - of an error. */ - -static void -reset_directive_searched (void *data) -{ - struct using_direct *direct = (struct using_direct *) data; - direct->searched = 0; -} - /* Search for NAME by applying all import statements belonging to BLOCK which are applicable in SCOPE. */ @@ -376,7 +366,6 @@ d_lookup_symbol_imports (const char *scope, const char *name, { struct using_direct *current; struct block_symbol sym; - struct cleanup *searched_cleanup; /* First, try to find the symbol in the given module. */ sym = d_lookup_symbol_in_module (scope, name, block, domain, 1); @@ -399,9 +388,8 @@ d_lookup_symbol_imports (const char *scope, const char *name, { /* Mark this import as searched so that the recursive call does not search it again. */ - current->searched = 1; - searched_cleanup = make_cleanup (reset_directive_searched, - current); + scoped_restore restore_searched + = make_scoped_restore (¤t->searched, 1); /* If there is an import of a single declaration, compare the imported declaration (after optional renaming by its alias) @@ -419,9 +407,6 @@ d_lookup_symbol_imports (const char *scope, const char *name, declaration, the search of this import is complete. */ if (sym.symbol != NULL || current->declaration) { - current->searched = 0; - discard_cleanups (searched_cleanup); - if (sym.symbol != NULL) return sym; @@ -433,10 +418,7 @@ d_lookup_symbol_imports (const char *scope, const char *name, if (strcmp (name, *excludep) == 0) break; if (*excludep) - { - discard_cleanups (searched_cleanup); - continue; - } + continue; /* If the import statement is creating an alias. */ if (current->alias != NULL) @@ -476,8 +458,6 @@ d_lookup_symbol_imports (const char *scope, const char *name, sym = d_lookup_symbol_in_module (current->import_src, name, block, domain, 1); } - current->searched = 0; - discard_cleanups (searched_cleanup); if (sym.symbol != NULL) return sym;