diff options
Diffstat (limited to 'compilerplugins/clang/unnecessaryparen.cxx')
-rw-r--r-- | compilerplugins/clang/unnecessaryparen.cxx | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/compilerplugins/clang/unnecessaryparen.cxx b/compilerplugins/clang/unnecessaryparen.cxx index fe3b0dd4b028..fdc83410cb0e 100644 --- a/compilerplugins/clang/unnecessaryparen.cxx +++ b/compilerplugins/clang/unnecessaryparen.cxx @@ -83,12 +83,10 @@ public: bool VisitCXXOperatorCallExpr(const CXXOperatorCallExpr *); bool TraverseUnaryExprOrTypeTraitExpr(UnaryExprOrTypeTraitExpr *); bool TraverseCaseStmt(CaseStmt *); - bool TraverseConditionalOperator(ConditionalOperator *); private: void VisitSomeStmt(Stmt const * stmt, const Expr* cond, StringRef stmtName); Expr const * insideSizeof = nullptr; Expr const * insideCaseStmt = nullptr; - Expr const * insideConditionalOperator = nullptr; }; bool UnnecessaryParen::TraverseUnaryExprOrTypeTraitExpr(UnaryExprOrTypeTraitExpr * expr) @@ -111,15 +109,6 @@ bool UnnecessaryParen::TraverseCaseStmt(CaseStmt * caseStmt) return ret; } -bool UnnecessaryParen::TraverseConditionalOperator(ConditionalOperator * conditionalOperator) -{ - auto old = insideConditionalOperator; - insideConditionalOperator = ignoreAllImplicit(conditionalOperator->getCond()); - bool ret = RecursiveASTVisitor::TraverseConditionalOperator(conditionalOperator); - insideConditionalOperator = old; - return ret; -} - bool UnnecessaryParen::VisitParenExpr(const ParenExpr* parenExpr) { if (ignoreLocation(parenExpr)) @@ -130,8 +119,6 @@ bool UnnecessaryParen::VisitParenExpr(const ParenExpr* parenExpr) return true; if (insideCaseStmt && parenExpr == insideCaseStmt) return true; - if (insideConditionalOperator && parenExpr == insideConditionalOperator) - return true; auto subExpr = ignoreAllImplicit(parenExpr->getSubExpr()); |