[PATCH] D35257: [clang-tidy] Add new modernize use unary assert check

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 11 07:13:56 PDT 2017


Eugene.Zelenko added a comment.

Please mention this check in docs/ReleaseNotes.rst (in alphabetical order).



================
Comment at: clang-tidy/modernize/UnaryStaticAssertCheck.cpp:28
+void UnaryStaticAssertCheck::check(const MatchFinder::MatchResult &Result) {
+
+  const auto *MatchedDecl =
----------------
Please remove empty line.


================
Comment at: docs/clang-tidy/checks/modernize-unary-static-assert.rst:15
+
+  void f_textless(int a){
+    static_assert(sizeof(a) <= 10, "");
----------------
Please Clang-format snippets. Same below.


================
Comment at: docs/clang-tidy/checks/modernize-unary-static-assert.rst:26
+  }
+
+
----------------
Please remove empty line.


Repository:
  rL LLVM

https://reviews.llvm.org/D35257





More information about the cfe-commits mailing list