Patchwork [C++,obvious?] PR 55632

login
register
mail settings
Submitter Paolo Carlini
Date Feb. 26, 2013, 10:42 a.m.
Message ID <512C918A.6090903@oracle.com>
Download mbox | patch
Permalink /patch/223190/
State New
Headers show

Comments

Paolo Carlini - Feb. 26, 2013, 10:42 a.m.
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.
Jason Merrill - Feb. 26, 2013, 1 p.m.
OK.

Jason

Patch

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,