From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21059 invoked by alias); 23 Jun 2011 21:33:01 -0000 Received: (qmail 21051 invoked by uid 22791); 23 Jun 2011 21:33:00 -0000 X-SWARE-Spam-Status: No, hits=-1.2 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,SPF_HELO_PASS,TW_CX,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from smtp-out.google.com (HELO smtp-out.google.com) (74.125.121.67) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 23 Jun 2011 21:32:46 +0000 Received: from hpaq1.eem.corp.google.com (hpaq1.eem.corp.google.com [172.25.149.1]) by smtp-out.google.com with ESMTP id p5NLWifs008197; Thu, 23 Jun 2011 14:32:44 -0700 Received: from gchare.mtv.corp.google.com (gchare.mtv.corp.google.com [172.18.111.122]) by hpaq1.eem.corp.google.com with ESMTP id p5NLWgkA017654; Thu, 23 Jun 2011 14:32:42 -0700 Received: by gchare.mtv.corp.google.com (Postfix, from userid 138564) id 898C71C375A; Thu, 23 Jun 2011 14:32:41 -0700 (PDT) To: reply@codereview.appspotmail.com, crowl@google.com, dnovillo@google.com, jason@redhat.com, gcc-patches@gcc.gnu.org Subject: Removed unused cp_binding_level field names_size. (issue4662052) Message-Id: <20110623213241.898C71C375A@gchare.mtv.corp.google.com> Date: Thu, 23 Jun 2011 21:35:00 -0000 From: gchare@google.com (Gabriel Charette) X-System-Of-Record: true X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2011-06/txt/msg01823.txt.bz2 The names_size member of cp_binding_level was write only. Removed it. Seems like it was introduced for java in 2002, but it's not used anywhere anymore in the code. Tested with bootstrap and full regression testing. 2011-06-23 Gabriel Charette * name-lookup.h (cp_binding_level): Removed unused member names_size. Update all users. diff --git a/gcc/cp/name-lookup.c b/gcc/cp/name-lookup.c index 953edd5..8bf5f5f 100644 --- a/gcc/cp/name-lookup.c +++ b/gcc/cp/name-lookup.c @@ -541,7 +541,6 @@ add_decl_to_level (tree decl, cxx_scope *b) necessary. */ TREE_CHAIN (decl) = b->names; b->names = decl; - b->names_size++; /* If appropriate, add decl to separate list of statics. We include extern variables because they might turn out to be diff --git a/gcc/cp/name-lookup.h b/gcc/cp/name-lookup.h index 009b5d9..5f266eb 100644 --- a/gcc/cp/name-lookup.h +++ b/gcc/cp/name-lookup.h @@ -191,9 +191,6 @@ struct GTY(()) cp_binding_level { are wrapped in TREE_LISTs; the TREE_VALUE is the OVERLOAD. */ tree names; - /* Count of elements in names chain. */ - size_t names_size; - /* A chain of NAMESPACE_DECL nodes. */ tree namespaces; -- This patch is available for review at http://codereview.appspot.com/4662052