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

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jul 9 06:03:37 PDT 2022


njames93 added inline comments.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:188
   by checking for an arbitrary expression in the second argument of ``memset``.
 
 - Improved :doc:`cppcoreguidelines-prefer-member-initializer
----------------
It should live here instead.


================
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.
+
----------------
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)


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