From a78145219359ff5c56b367e329cd0a174bd8a27b Mon Sep 17 00:00:00 2001 From: Argyrios Kyrtzidis <akyrtzi@gmail.com> Date: Tue, 24 Jun 2008 22:31:41 +0000 Subject: [PATCH] The only caller of this knows that the current token is l_brace, so this can be an assert; suggestion by Chris. llvm-svn: 52696 --- clang/lib/Parse/ParseCXXInlineMethods.cpp | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/clang/lib/Parse/ParseCXXInlineMethods.cpp b/clang/lib/Parse/ParseCXXInlineMethods.cpp index b593f8966818..ca0003440fe0 100644 --- a/clang/lib/Parse/ParseCXXInlineMethods.cpp +++ b/clang/lib/Parse/ParseCXXInlineMethods.cpp @@ -23,21 +23,10 @@ Parser::DeclTy * Parser::ParseCXXInlineMethodDef(AccessSpecifier AS, Declarator &D) { assert(D.getTypeObject(0).Kind == DeclaratorChunk::Function && "This isn't a function declarator!"); + assert(Tok.is(tok::l_brace) && "Current token not a '{'!"); DeclTy *FnD = Actions.ActOnCXXMemberDeclarator(CurScope, AS, D, 0, 0, 0); - // We should have an opening brace now. - if (Tok.isNot(tok::l_brace)) { - Diag(Tok, diag::err_expected_fn_body); - - // Skip over garbage, until we get to '{'. Don't eat the '{'. - SkipUntil(tok::l_brace, true, true); - - // If we didn't find the '{', bail out. - if (Tok.isNot(tok::l_brace)) - return FnD; - } - // Consume the tokens and store them for later parsing. getCurTopClassStack().push(LexedMethod(FnD));