Patchwork [Obj-C++] Found a small paste-o in parser.c?

login
register
mail settings
Submitter Ed Smith-Rowland
Date Nov. 11, 2012, 4:10 a.m.
Message ID <509F2522.9040803@verizon.net>
Download mbox | patch
Permalink /patch/198236/
State New
Headers show

Comments

Ed Smith-Rowland - Nov. 11, 2012, 4:10 a.m.
I found this suspicious looking line in cp/parser.c () while looking at 
__thread and thread_local.

Look at the patterns of the if blocks above the line in question to verify.

Built and tested on x86_64-linux.

Ed
2012-11-11  Ed Smith-Rowland  <3dw4rd@verizon.net>

	* parser.c (cp_parser_objc_class_ivars):
	Index declspecs.locations by ds_typedef rather than ds_thread.
Jason Merrill - Nov. 12, 2012, 5:05 a.m.
OK.

Jason

Patch

Index: parser.c
===================================================================
--- parser.c	(revision 193380)
+++ parser.c	(working copy)
@@ -24650,7 +24650,7 @@ 
       if (decl_spec_seq_has_spec_p (&declspecs, ds_typedef))
 	{
 	  cp_parser_error (parser, "invalid type for instance variable");
-	  declspecs.locations[ds_thread] = 0;
+	  declspecs.locations[ds_typedef] = 0;
 	}
 
       prefix_attributes = declspecs.attributes;