[PATCH] D132461: [clang-tidy] Add cppcoreguidelines-avoid-do-while check

[Eugene Zelenko via Phabricator via cfe-commits]

Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/avoid-do-while.rst:6
+
+This check warns when using ``do-while`` loops. They are less readable than
+plain ``while`` loops since the termination condition is at the end and the
----------------
Please synchronize first statement with Release Notes. `This check` us usually omitted.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/avoid-do-while.rst:21
+
+The check implements
+`rule ES.75 of C++ Core Guidelines <https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#Res-do>`_.
----------------
Links usually placed at the end.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/avoid-do-while.rst:41
+
+  Defaults to ``false``.
----------------
Please use single back-ticks for options values.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132461/new/

https://reviews.llvm.org/D132461