[PATCH] D64671: [clang-tidy] New check: misc-init-local-variables
Eugene Zelenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Aug 24 08:07:49 PDT 2019
Eugene.Zelenko added inline comments.
================
Comment at: clang-tools-extra/test/clang-tidy/cppcoreguidelines-init-variables.cpp:3
+
+#include <stdint.h>
+
----------------
lebedev.ri wrote:
> Eugene.Zelenko wrote:
> > It'll be better to include cstdint.
> I'll correct that comment: tests normally shouldn't depend on system headers; can you just mock whatever you need from there?
If test is C++, why not to include C++ equivalent of system headers? See modernize-deprecated-headers.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64671/new/
https://reviews.llvm.org/D64671
More information about the cfe-commits
mailing list