diff options
author | Eugene Zelenko <eugene.zelenko@gmail.com> | 2016-02-12 22:53:10 +0000 |
---|---|---|
committer | Eugene Zelenko <eugene.zelenko@gmail.com> | 2016-02-12 22:53:10 +0000 |
commit | 1ced5095e4c621373ec4d24927216af67a28fe37 (patch) | |
tree | 8303fc76bead3a0ac79d1df61e095bc34f44ee0b /clang/lib/Parse/ParseObjc.cpp | |
parent | b534ce46bd40eaa909a20f40ae576a2627da4f90 (diff) | |
download | bcm5719-llvm-1ced5095e4c621373ec4d24927216af67a28fe37.tar.gz bcm5719-llvm-1ced5095e4c621373ec4d24927216af67a28fe37.zip |
Fix remaining Clang-tidy readability-redundant-control-flow warnings; other minor fixes.
Differential revision: http://reviews.llvm.org/D17218
llvm-svn: 260757
Diffstat (limited to 'clang/lib/Parse/ParseObjc.cpp')
-rw-r--r-- | clang/lib/Parse/ParseObjc.cpp | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/clang/lib/Parse/ParseObjc.cpp b/clang/lib/Parse/ParseObjc.cpp index 6a680907cce..c30ec1d7b78 100644 --- a/clang/lib/Parse/ParseObjc.cpp +++ b/clang/lib/Parse/ParseObjc.cpp @@ -21,6 +21,7 @@ #include "clang/Sema/Scope.h" #include "llvm/ADT/SmallVector.h" #include "llvm/ADT/StringExtras.h" + using namespace clang; /// Skips attributes after an Objective-C @ directive. Emits a diagnostic. @@ -99,16 +100,20 @@ class Parser::ObjCTypeParamListScope { Sema &Actions; Scope *S; ObjCTypeParamList *Params; + public: ObjCTypeParamListScope(Sema &Actions, Scope *S) : Actions(Actions), S(S), Params(nullptr) {} + ~ObjCTypeParamListScope() { leave(); } + void enter(ObjCTypeParamList *P) { assert(!Params); Params = P; } + void leave() { if (Params) Actions.popObjCTypeParamList(S, Params); @@ -1961,7 +1966,6 @@ void Parser::ParseObjCClassInstanceVariables(Decl *interfaceDecl, } HelperActionsForIvarDeclarations(interfaceDecl, atLoc, T, AllIvarDecls, false); - return; } /// objc-protocol-declaration: @@ -2938,7 +2942,6 @@ bool Parser::isStartOfObjCClassMessageMissingOpenBracket() { InMessageExpression) return false; - ParsedType Type; if (Tok.is(tok::annot_typename)) @@ -3567,7 +3570,7 @@ ExprResult Parser::ParseObjCSelectorExpression(SourceLocation AtLoc) { T.getOpenLocation(), T.getCloseLocation(), !HasOptionalParen); - } +} void Parser::ParseLexedObjCMethodDefs(LexedMethod &LM, bool parseMethod) { // MCDecl might be null due to error in method or c-function prototype, etc. @@ -3623,6 +3626,4 @@ void Parser::ParseLexedObjCMethodDefs(LexedMethod &LM, bool parseMethod) { while (Tok.getLocation() != OrigLoc && Tok.isNot(tok::eof)) ConsumeAnyToken(); } - - return; } |