[PATCH] D51949: [clang-tidy] new check 'readability-isolate-declaration'

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 28 09:02:33 PDT 2018


JonasToth added inline comments.


================
Comment at: docs/clang-tidy/checks/readability-isolate-declaration.rst:33
+  for (int Begin = 0, End = 100; Begin < End; ++Begin);
+  it (int Begin = 42, Result = some_function(Begin); Begin == Result);
+
----------------
typo
mention `if () int i, j, k = function();` as well


================
Comment at: docs/clang-tidy/checks/readability-isolate-declaration.rst:87
+    int Unconditional,
+    #if CONFIGURATION
+        IfConfigured = 42,
----------------
reduce indendation for PP directives


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D51949





More information about the cfe-commits mailing list