From patchwork Tue Feb 26 10:42:18 2013
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [C++,obvious?] PR 55632
Date: Tue, 26 Feb 2013 00:42:18 -0000
From: Paolo Carlini <paolo.carlini@oracle.com>
X-Patchwork-Id: 223190
Message-Id: <512C918A.6090903@oracle.com>
To: "gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>
Cc: Jason Merrill <jason@redhat.com>

Hi,

I was wondering whether we want to apply as obvious the below (given the 
cp_storage_class enum) and resolve the PR.

Thanks,
Paolo.

////////////////////////
2013-02-26  David Binderman  <dcb314@hotmail.com>

	PR c++/55632
	* decl.c (grokdeclarator): Tidy publicp assignment.


Index: decl.c
===================================================================
--- decl.c	(revision 196276)
+++ decl.c	(working copy)
@@ -10634,9 +10634,8 @@ grokdeclarator (const cp_declarator *declarator,
 	      }
 	  }
 
-	/* Record presence of `static'.  */
+	/* Record whether the function is public.  */
 	publicp = (ctype != NULL_TREE
-		   || storage_class == sc_extern
 		   || storage_class != sc_static);
 
 	decl = grokfndecl (ctype, type, original_name, parms, unqualified_id,