From 1453e310fd340b21fde4cb4b9f02956f0486f630 Mon Sep 17 00:00:00 2001 From: Richard Smith Date: Tue, 27 Mar 2012 01:42:32 +0000 Subject: Add cross-referencing comments to ParseDirectDeclarator to note that isConstructorDeclaration also needs updating for any extension to the grammar of a direct-declarator. llvm-svn: 153490 --- clang/lib/Parse/ParseDecl.cpp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'clang/lib/Parse/ParseDecl.cpp') diff --git a/clang/lib/Parse/ParseDecl.cpp b/clang/lib/Parse/ParseDecl.cpp index 75abcae2828..9d8aed53331 100644 --- a/clang/lib/Parse/ParseDecl.cpp +++ b/clang/lib/Parse/ParseDecl.cpp @@ -3393,7 +3393,8 @@ bool Parser::isConstructorDeclarator() { ConsumeToken(); // If this is not a constructor, we must be parsing a declarator, - // which must have one of the following syntactic forms: + // which must have one of the following syntactic forms (see the + // grammar extract at the start of ParseDirectDeclarator): switch (Tok.getKind()) { case tok::l_paren: // C(X ( int)); @@ -3543,7 +3544,8 @@ void Parser::ParseDeclarator(Declarator &D) { /// ptr-operator production. /// /// If the grammar of this construct is extended, matching changes must also be -/// made to TryParseDeclarator and MightBeDeclarator. +/// made to TryParseDeclarator and MightBeDeclarator, and possibly to +/// isConstructorDeclarator. /// /// declarator: [C99 6.7.5] [C++ 8p4, dcl.decl] /// [C] pointer[opt] direct-declarator @@ -3735,6 +3737,8 @@ void Parser::ParseDeclaratorInternal(Declarator &D, /// '~' class-name /// template-id /// +/// Note, any additional constructs added here may need corresponding changes +/// in isConstructorDeclarator. void Parser::ParseDirectDeclarator(Declarator &D) { DeclaratorScopeObj DeclScopeObj(*this, D.getCXXScopeSpec()); -- cgit v1.2.3