[PATCH] D129420: [clang-tidy] Initialize boolean variables with 'false' in cppcoreguidelines-init-variables' fix-it

Danny Mösch via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jul 9 06:48:32 PDT 2022


SimplyDanny marked an inline comment as done.
SimplyDanny added inline comments.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:250-253
+- Made the fix-it of :doc:`cppcoreguidelines-init-variables
+  <clang-tidy/checks/cppcoreguidelines/init-variables>` use ``false`` to initialize
+  boolean variables.
+
----------------
njames93 wrote:
> Sorry I should've mentioned, we keep this list in alphabetical order(for the names of the checks).
> Can you push a patch to move this into its correct place(no need for a review and commit as NFC)
Ah sorry. I could have noticed that. Fixed in [[ https://github.com/llvm/llvm-project/commit/963192228b05e3bce86cf520e2bda5f069be9894 | this commit ]].


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129420



More information about the cfe-commits mailing list