[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