support arbitrary order of type-specifier and storage-class-specifier

TODO: we really need to adhere to the spec better by adding a
      parserDeclSpecifier which should solve this problem and others

BUG: 253827

languages/cpp/parser/parser.cpp

@@ -3478,6 +3478,8 @@ bool Parser::parseLabeledStatement(StatementAST *&node)
   return false;
 }
 
+///TODO: decl-specifier-seq is currently not used at all,
+///      can lead to problems for certain orders of specifiers
 bool Parser::parseBlockDeclaration(DeclarationAST *&node)
 {
   switch(session->token_stream->lookAhead())
@@ -3518,6 +3520,9 @@ bool Parser::parseBlockDeclaration(DeclarationAST *&node)
       return false;
     }
 
+  if (!storageSpec) // see also https://bugs.kde.org/show_bug.cgi?id=253827
+    parseStorageClassSpecifier(storageSpec);
+
   parseCvQualify(cv);
   spec->cv = cv;
 

languages/cpp/parser/tests/test_parser.cpp

@@ -811,6 +811,15 @@ void TestParser::testTypeID()
   QVERIFY(control.problems().isEmpty());
 }
 
+void TestParser::testRegister()
+{
+  // see also: http://bugsfiles.kde.org/attachment.cgi?id=61647
+  QString code = "void foo() { register int i; int register j; }\n";
+  TranslationUnitAST* ast = parse(code.toLocal8Bit());
+  dumper.dump(ast, lastSession->token_stream);
+  QVERIFY(control.problems().isEmpty());
+}
+
 TranslationUnitAST* TestParser::parse(const QByteArray& unit)
 {
   control = Control(); // Clear the problems

languages/cpp/parser/tests/test_parser.h

@@ -81,6 +81,8 @@ private slots:
   void testTypeID_data();
   void testTypeID();
 
+  void testRegister();
+
   //BEGIN C++2011 support
   void testRangeBasedFor();
   void testRValueReference();