[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