Eugene.Zelenko added a comment.
Please mention this check in docs/ReleaseNotes.rst (in alphabetical order).
Will be good idea to run Clang-tidy modernize and readability checks over new
code.
================
Comment at: clang-tidy/readability/CompoundStatementSizeCheck.cpp:41
+ ++Info.Branches;
+ // fallthrough
+ case Stmt::CompoundStmtClass:
----------------
Please use LLVM_FALLTHROUGH instead (defined in llvm/Support/Compiler.h)
================
Comment at: clang-tidy/readability/CompoundStatementSizeCheck.cpp:64
+ struct CompoundStatementInfo {
+ CompoundStatementInfo() : Lines(0), Statements(0), Branches(0) {}
+ unsigned Lines;
----------------
Please use default member initializers and = default;
https://reviews.llvm.org/D31252
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
